]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
iprep: fix parsing ip-rep data with carriage return 9426/head
authorThomas Winter <Thomas.Winter@alliedtelesis.co.nz>
Mon, 15 May 2023 02:18:47 +0000 (14:18 +1200)
committerThomas Winter <Thomas.Winter@alliedtelesis.co.nz>
Thu, 31 Aug 2023 23:15:31 +0000 (11:15 +1200)
Commit e7c0f0ad91fd removed uses of atoi with a new number parsing
functions. This broke parsing ip-reputation data files that contained
trailing carriage returns as it was being included in the number
string to convert.

Bug: #6243.

src/reputation.c

index b9f2186d0100a89fb1b57531efd1d4a1470d39dc..75f3ba0c3fa41209c0a86bcece70edce9b881cb0 100644 (file)
@@ -282,7 +282,8 @@ static int SRepSplitLine(SRepCIDRTree *cidr_ctx, char *line, Address *ip, uint8_
     char *origline = line;
 
     while (i < (int)line_len) {
-        if (line[i] == ',' || line[i] == '\n' || line[i] == '\0' || i == (int)(line_len - 1)) {
+        if (line[i] == ',' || line[i] == '\n' || line[i] == '\r' || line[i] == '\0' ||
+                i == (int)(line_len - 1)) {
             line[i] = '\0';
 
             ptrs[idx] = line;