discover: Release process resources on error

Fixes Coverity defect #30472

Signed-off-by: Sam Mendoza-Jonas <sam@mendozajonas.com>

diff --git a/discover/device-handler.c b/discover/device-handler.c
index 77f9e3c59b9bb761d02a668f26ad130187f516d9..9de2a1901ea16b41f1f78513bec59cfbfc302f87 100644 (file)
--- a/discover/device-handler.c
+++ b/discover/device-handler.c
@@ -1099,10 +1099,12 @@ static char *device_from_addr(void *ctx, struct pb_url *url)
 
        rc = process_run_sync(p);
 
 
        rc = process_run_sync(p);
 

-       if (rc) {
+       if (rc || p->exit_status) {

                /* ip has complained for some reason; most likely
                 * there is no route to the host - bail out */
                /* ip has complained for some reason; most likely
                 * there is no route to the host - bail out */

-               pb_debug("%s: No route to %s\n",__func__,url->host);
+               pb_debug("%s: `ip` returns non-zero exit status\n", __func__);
+               pb_debug("ip buf: %s\n", p->stdout_buf);
+               process_release(p);

                return NULL;
        }
 
                return NULL;
        }