Null-pointer dereference in PIDS table processing fixed

mezantrop · Sep 30, 2024 · 06e2e9d · 06e2e9d
1 parent 408744a
commit 06e2e9d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 # CHANGELOG
 
-* **2024.09.28    Current**
+* **2024.09.30    Current**
+  * `ts-warp.c`: Null-pointer dereference in `PIDS` table processing fixed
   * `ts-warp.c`: `ACT` reporting `[SECTIONS]`
   * `inifile.c`: Correct usage of multiple links of a proxy-chain
   * `inifile.c`: Potential buffer-overflow fixed in parsing `[SECTIONS]`

diff --git a/ts-warp.c b/ts-warp.c
@@ -607,8 +607,10 @@ All parameters are optional:
             push_ini = NULL;
             if (!c->section_name || strlen(c->section_name) == 0) {
                 tmp_daddr.ip_addr = c->traffic.daddr;
-                push_ini = ini_look_server(ini_root, tmp_daddr);
-                free(c->section_name); c->section_name = strdup(push_ini->section_name);
+                if ((push_ini = ini_look_server(ini_root, tmp_daddr))) {
+                    free(c->section_name);
+                    c->section_name = strdup(push_ini->section_name);
+                }
             }
 
             if (c == pids && c->status >= 0) {                              /* Remove pidlist root entry */