Hi, I am just guessing what this was supposed to mean, but I am pretty sure that a NULL pointer dereference is not a good idea. Untested. Florian diff --git a/udev/udev-rules.c b/udev/udev-rules.c index 53baa18..ee009b1 100644 --- a/udev/udev-rules.c +++ b/udev/udev-rules.c @@ -875,7 +875,7 @@ static int get_key(struct udev *udev, char **line, char **key, enum operation_ty char *temp; linepos = *line; - if (linepos == NULL && linepos[0] == '\0') + if (linepos == NULL || linepos[0] == '\0') return -1; /* skip whitespace */ -- To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
