From 1822e9b033fcd91a7dfd3b0bfd12cfbbd93fc470 Mon Sep 17 00:00:00 2001 From: Kay Sievers Date: Sun, 26 Oct 2008 02:31:54 +0100 Subject: [PATCH] do not init string arrays, just clear first byte --- extras/edd_id/edd_id.c | 5 +++-- extras/scsi_id/scsi_id.c | 3 ++- udev/udev-event.c | 3 ++- udev/udev-node.c | 6 ++++-- udev/udev-rules.c | 3 ++- 5 files changed, 13 insertions(+), 7 deletions(-) diff --git a/extras/edd_id/edd_id.c b/extras/edd_id/edd_id.c index d7aec1488..4154387cb 100644 --- a/extras/edd_id/edd_id.c +++ b/extras/edd_id/edd_id.c @@ -45,9 +45,9 @@ int main(int argc, char *argv[]) int sysfs_fd; DIR *dir = NULL; int rc = 1; - char match[NAME_MAX] = ""; + char match[NAME_MAX]; - udev = udev_new(); + udev = udev_new(); if (udev == NULL) goto exit; @@ -124,6 +124,7 @@ int main(int argc, char *argv[]) } /* lookup signature in sysfs to determine the name */ + match[0] = '\0'; for (dent = readdir(dir); dent != NULL; dent = readdir(dir)) { char file[UTIL_PATH_SIZE]; char sysfs_id_buf[256]; diff --git a/extras/scsi_id/scsi_id.c b/extras/scsi_id/scsi_id.c index 86c76cfcc..45e48601f 100644 --- a/extras/scsi_id/scsi_id.c +++ b/extras/scsi_id/scsi_id.c @@ -579,8 +579,9 @@ static int scsi_id(struct udev *udev, char *maj_min_dev) struct scsi_id_device dev_scsi; int good_dev; int page_code; - char serial_short[MAX_SERIAL_LEN] = ""; + char serial_short[MAX_SERIAL_LEN]; + serial_short[0] = '\0'; set_inq_values(udev, &dev_scsi, maj_min_dev); /* get per device (vendor + model) options from the config file */ diff --git a/udev/udev-event.c b/udev/udev-event.c index fcb999370..e297df038 100644 --- a/udev/udev-event.c +++ b/udev/udev-event.c @@ -291,9 +291,10 @@ found: err(event->udev, "missing file parameter for attr\n"); else { const char *val; - char value[UTIL_NAME_SIZE] = ""; + char value[UTIL_NAME_SIZE]; size_t size; + value[0] = '\0'; util_resolve_subsys_kernel(event->udev, attr, value, sizeof(value), 1); val = udev_device_get_sysattr_value(event->dev, attr); diff --git a/udev/udev-node.c b/udev/udev-node.c index b39100dce..e402dc9de 100644 --- a/udev/udev-node.c +++ b/udev/udev-node.c @@ -143,7 +143,7 @@ exit: static int node_symlink(struct udev *udev, const char *node, const char *slink) { struct stat stats; - char target[UTIL_PATH_SIZE] = ""; + char target[UTIL_PATH_SIZE]; char slink_tmp[UTIL_PATH_SIZE + sizeof(TMP_FILE_EXT)]; int i = 0; int tail = 0; @@ -151,6 +151,7 @@ static int node_symlink(struct udev *udev, const char *node, const char *slink) int err = 0; /* use relative link */ + target[0] = '\0'; while (node[i] && (node[i] == slink[i])) { if (node[i] == '/') tail = i+1; @@ -272,7 +273,7 @@ static int update_link(struct udev_device *dev, const char *slink, int test) struct udev *udev = udev_device_get_udev(dev); struct udev_list_node dev_list; struct udev_list_entry *dev_entry; - char target[UTIL_PATH_SIZE] = ""; + char target[UTIL_PATH_SIZE]; int count; int priority = 0; int rc = 0; @@ -294,6 +295,7 @@ static int update_link(struct udev_device *dev, const char *slink, int test) } /* find the device with the highest priority */ + target[0] = '\0'; udev_list_entry_foreach(dev_entry, udev_list_get_entry(&dev_list)) { const char *syspath; struct udev_device *dev_db; diff --git a/udev/udev-rules.c b/udev/udev-rules.c index 1647f5cb0..a6f922035 100644 --- a/udev/udev-rules.c +++ b/udev/udev-rules.c @@ -547,11 +547,12 @@ static int wait_for_file(struct udev_device *dev, const char *file, int timeout) { struct udev *udev = udev_device_get_udev(dev); char filepath[UTIL_PATH_SIZE]; - char devicepath[UTIL_PATH_SIZE] = ""; + char devicepath[UTIL_PATH_SIZE]; struct stat stats; int loop = timeout * WAIT_LOOP_PER_SECOND; /* a relative path is a device attribute */ + devicepath[0] = '\0'; if (file[0] != '/') { util_strlcpy(devicepath, udev_get_sys_path(udev), sizeof(devicepath)); util_strlcat(devicepath, udev_device_get_devpath(dev), sizeof(devicepath)); -- 2.30.2