From: kay.sievers@vrfy.org Date: Mon, 28 Mar 2005 10:37:54 +0000 (+0200) Subject: [PATCH] check for strlen()==0 before accessing strlen()-1 X-Git-Tag: 057~19 X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=bb1a77d34cd004b44b30a79d7521957c3881e497 [PATCH] check for strlen()==0 before accessing strlen()-1 --- diff --git a/udev_rules.c b/udev_rules.c index b7905477d..c07abb9be 100644 --- a/udev_rules.c +++ b/udev_rules.c @@ -594,15 +594,16 @@ static int match_rule(struct udevice *udev, struct udev_rule *rule, for (i = 0; i < rule->sysfs_pair_count; i++) { struct key_pair *pair; char value[VALUE_SIZE]; + size_t len; pair = &rule->sysfs_pair[i]; if (find_sysfs_attribute(class_dev, parent_device, pair->name, value, sizeof(value)) != 0) goto try_parent; /* strip trailing whitespace of value, if not asked to match for it */ - if (!isspace(pair->value[strlen(pair->value)-1])) { - size_t len = strlen(value); - + len = strlen(pair->value); + if (len && !isspace(pair->value[len-1])) { + len = strlen(value); while (len > 0 && isspace(value[len-1])) value[--len] = '\0'; dbg("removed %i trailing whitespace chars from '%s'", strlen(value)-len, value);