]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
Fix potential iprep file parsing issue.
authorVictor Julien <victor@inliniac.net>
Wed, 6 Mar 2013 14:32:53 +0000 (15:32 +0100)
committerVictor Julien <victor@inliniac.net>
Wed, 6 Mar 2013 14:32:53 +0000 (15:32 +0100)
src/reputation.c

index 02e41b14a0176cb9ee1a3136ea9da705d74c6527..fec8a08746d49825b889a31af8eb41363fda5f0a 100644 (file)
@@ -245,6 +245,8 @@ int SRepLoadCatFile(char *filename) {
 
     while(fgets(line, (int)sizeof(line), fp) != NULL) {
         size_t len = strlen(line);
+        if (len == 0)
+            continue;
 
         /* ignore comments and empty lines */
         if (line[0] == '\n' || line [0] == '\r' || line[0] == ' ' || line[0] == '#' || line[0] == '\t')
@@ -254,7 +256,10 @@ int SRepLoadCatFile(char *filename) {
 
         /* Check if we have a trailing newline, and remove it */
         len = strlen(line);
-        if (len > 0 && (line[len - 1] == '\n' || line[len - 1] == '\r')) {
+        if (len == 0)
+            continue;
+
+        if (line[len - 1] == '\n' || line[len - 1] == '\r') {
             line[len - 1] = '\0';
         }