From 467a04de7d615dac943c051c27b851316c198bde Mon Sep 17 00:00:00 2001
From: Dimitri Staessens <dimitri@ouroboros.rocks>
Date: Sun, 21 Jan 2024 12:19:35 +0100
Subject: ipcpd: Fix memleak in rib_readdir

On fail_dup, the last element (0) was not free'd.

Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks>
Signed-off-by: Sander Vrijders <sander@ouroboros.rocks>
---
 src/ipcpd/ipcp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/ipcpd/ipcp.c b/src/ipcpd/ipcp.c
index 61c811c3..312c4a79 100644
--- a/src/ipcpd/ipcp.c
+++ b/src/ipcpd/ipcp.c
@@ -183,8 +183,8 @@ static int ipcp_rib_readdir(char *** buf)
 
         return i;
  fail_dup:
-        while (--i > 0)
-                free((*buf)[i]);
+        while (i > 0)
+                free((*buf)[--i]);
  fail:
         free(*buf);
 
-- 
cgit v1.2.3