Coverity: points out that the normal return path does a double free

In b31fc3e where I added the error paths, all the errors are handled correctly - what I never tested was the normal/sucessful path. On the normal/sucessful path, it does a double free on dev_names (which is a bug). This fixed that by returning in the correct place during a sucessful quit. Signed-off-by: Robin Getz <[email protected]>
analogdevicesinc · Apr 27, 2020 · 7f0bcd0 · 7f0bcd0
1 parent a613dd0
commit 7f0bcd0
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/examples/iio-monitor.c b/examples/iio-monitor.c
@@ -398,6 +398,8 @@ static void show_main_screen(struct iio_context *ctx)
 	free(dev_names);
 	destroyCDKScreen(screen);
 
+	return;
+
 dev_name_err:
 	for (i = 0; i < nb_devices; i++) {
 		if (dev_names[i])