Null pointer defref in adding info
authorEzra Peisach <epeisach@mit.edu>
Tue, 7 Apr 2009 17:57:56 +0000 (17:57 +0000)
committerEzra Peisach <epeisach@mit.edu>
Tue, 7 Apr 2009 17:57:56 +0000 (17:57 +0000)
Clearly the code is broken - and we either never use it - or callers never pass
NULL...

Detected by clang static checker.

ticket: 6442

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@22172 dc483132-0cff-0310-8789-dd5450dbe970

src/util/ss/help.c

index 10b1c10756105765ce418908951fc268f479b52b..981a75fabde20a834974d760dd0d17df55badb49 100644 (file)
@@ -107,7 +107,7 @@ void ss_add_info_dir(sci_idx, info_dir, code_ptr)
     register char **dirs;
 
     info = ss_info(sci_idx);
-    if (info_dir == NULL && *info_dir) {
+    if ((info_dir == NULL) || (*info_dir == '\0')) {
        *code_ptr = SS_ET_NO_INFO_DIR;
        return;
     }