From f2fe5da31deebc689bb64e71c8e873efc925d312 Mon Sep 17 00:00:00 2001 From: Eric Sandeen Date: Tue, 24 Feb 2009 15:13:39 -0600 Subject: e2fsprogs: fix potential null ptr defef in check_for_modules() The coverity scanner found this one. If a line in modules.dep has a ":" but no "/" then: if ((cp = strchr(buf, ':')) != NULL) *cp = 0; else continue; if ((cp = strrchr(buf, '/')) != NULL) cp++; /* XXX else cp is still null */ i = strlen(cp); ... we will deref a null pointer (cp). This can be demonstrated by putting a line like: foo.ko: into modules.dep. The below change just says that if no "/" is found, treat the whole string as the module name. Addresses-Red-Hat-Bugzilla: #486997 Signed-off-by: Eric Sandeen Signed-off-by: "Theodore Ts'o" --- lib/blkid/probe.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'lib/blkid') diff --git a/lib/blkid/probe.c b/lib/blkid/probe.c index 17feb9df..91a63131 100644 --- a/lib/blkid/probe.c +++ b/lib/blkid/probe.c @@ -227,6 +227,8 @@ static int check_for_modules(const char *fs_name) continue; if ((cp = strrchr(buf, '/')) != NULL) cp++; + else + cp = buf; i = strlen(cp); if (i > 3) { t = cp + i - 3; -- cgit v1.2.3