From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:59106) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJjKA-00029J-P9 for qemu-devel@nongnu.org; Thu, 04 Oct 2012 07:10:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TJjK4-0002zy-Of for qemu-devel@nongnu.org; Thu, 04 Oct 2012 07:10:38 -0400 Received: from mx.meyering.net ([88.168.87.75]:56991 helo=hx.meyering.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJjK4-0002y0-BM for qemu-devel@nongnu.org; Thu, 04 Oct 2012 07:10:32 -0400 From: Jim Meyering Date: Thu, 4 Oct 2012 13:09:43 +0200 Message-Id: <1349349003-15672-1-git-send-email-jim@meyering.net> Subject: [Qemu-devel] [PATCHv3 00/20] strncpy: best avoided List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jim Meyering From: Jim Meyering I included a quick classification of these change sets for the original series, (see https://lists.gnu.org/archive/html/qemu-devel/2012-05/msg01151.html) rebased in late May but perhaps the release-related timing was off. I've rebased one last time to prepare this v3 series. Other than the rebase, which ended up obviating 1 or 2 commits, there have has been no change since v2. Jim Meyering (20): scsi,pci,qdev,isa-bus,sysbus: don't let *_get_fw_dev_path return NULL sparc: use g_strdup in place of unchecked strdup block: avoid buffer overrun by using pstrcpy, not strncpy sheepdog: avoid a few buffer overruns vmdk: relative_path: use pstrcpy in place of strncpy hw/9pfs: avoid buffer overrun lm32: avoid buffer overrun os-posix: avoid buffer overrun ppc: avoid buffer overrun: use pstrcpy, not strncpy linux-user: remove two unchecked uses of strdup ui/vnc: simplify and avoid strncpy bt: replace fragile snprintf use and unwarranted strncpy virtio-9p: avoid unwarranted uses of strncpy vscsi: avoid unwarranted strncpy qemu-ga: prefer pstrcpy: consistently NUL-terminate ifreq.ifr_name libcacard/vcard_emul_nss: use pstrcpy in place of strncpy acpi: remove strzcpy (strncpy-identical) function; just use strncpy qcow2: mark this file's sole strncpy use as justified hw/r2d: add comment: this strncpy use is ok doc: update HACKING wrt strncpy/pstrcpy HACKING | 9 +++++---- block.c | 5 +++-- block/qcow2.c | 1 + block/sheepdog.c | 34 ++++++++++++++++++++++------------ block/vmdk.c | 3 +-- hw/9pfs/virtio-9p-posix-acl.c | 6 ++++-- hw/9pfs/virtio-9p-synth.c | 4 ++-- hw/9pfs/virtio-9p-xattr-user.c | 3 ++- hw/9pfs/virtio-9p-xattr.c | 3 ++- hw/acpi.c | 24 ++++++++---------------- hw/bt-hci.c | 7 ++----- hw/ide/qdev.c | 2 +- hw/isa-bus.c | 2 +- hw/lm32_hwsetup.h | 2 +- hw/pci.c | 2 +- hw/qdev.c | 2 +- hw/r2d.c | 2 ++ hw/scsi-bus.c | 8 ++------ hw/spapr_vscsi.c | 2 +- hw/sysbus.c | 2 +- libcacard/Makefile | 3 +++ libcacard/vcard_emul_nss.c | 3 +-- linux-user/elfload.c | 12 +++++++----- os-posix.c | 3 +-- qga/commands-posix.c | 2 +- target-ppc/kvm.c | 2 +- target-sparc/cpu.c | 4 ++-- ui/vnc-auth-sasl.c | 4 +--- 28 files changed, 80 insertions(+), 76 deletions(-) -- 1.8.0.rc0.18.gf84667d