Svace Bug Fix

Samsung · Aug 29, 2017 · 5205be2 · 5205be2
1 parent 602de0e
commit 5205be2
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 9 deletions.
diff --git a/apps/examples/easysetup/easysetup_main.c b/apps/examples/easysetup/easysetup_main.c
@@ -261,11 +261,11 @@ static void http_post_ctrl_cb(struct http_client_t *client, struct http_req_mess
 			}
 		}
 		cJSON_Delete(request_json);
-	}
 
-	if (msg->encoding == HTTP_CHUNKED_ENCODING && msg->entity[0] != '\0') {
-		g_stop_easysetup = stop_val;
-		return;
+		if (msg->encoding == HTTP_CHUNKED_ENCODING && msg->entity[0] != '\0') {
+			g_stop_easysetup = stop_val;
+			return;
+		}
 	}
 
 	http_send_response(client, 200, NULL, NULL);

diff --git a/apps/examples/iperf/iperf_main.c b/apps/examples/iperf/iperf_main.c
@@ -128,7 +128,6 @@ int iperf_main(int argc, char **argv)
 	}
 
 	if (run(test) < 0) {
-		iperf_free_test(test);
 		printf("error - %s\n", iperf_strerror(i_errno));
 		goto main_exit;
 	}
@@ -164,9 +163,9 @@ static int run(struct iperf_test *test)
 	case 'c':
 		if (iperf_run_client(test) < 0) {
 			close(test->ctrl_sck);
+			iperf_client_end(test);
 			iperf_free_test(test);
 			printf("error - %s\n", iperf_strerror(i_errno));
-			iperf_client_end(test);
 			goto run_exit;
 		}
 		break;

diff --git a/apps/netutils/dhcpc/dhcpc.c b/apps/netutils/dhcpc/dhcpc.c
@@ -393,7 +393,7 @@ void *dhcpc_open(const char *intf)
 
 	pdhcpc->sockfd = socket(PF_INET, SOCK_DGRAM, 0);
 	if (pdhcpc->sockfd < 0) {
-		nvdbg("socket handle %d\n", ret);
+		nvdbg("get the socket handle fail\n");
 		free(pdhcpc);
 		return NULL;
 	}

diff --git a/os/net/tls/easy_tls.c b/os/net/tls/easy_tls.c
@@ -279,7 +279,6 @@ tls_ctx *TLSCtx(tls_cred *cred)
 	return ctx;
 errout:
 	TLSCtx_free(ctx);
-	TLS_FREE(ctx);
 
 	return NULL;
 }
@@ -381,7 +380,7 @@ tls_session *TLSSession(int fd, tls_ctx *ctx, tls_opt *opt)
 	return session;
 errout:
 	TLSSession_free(session);
-	TLS_FREE(session);
+
 	return NULL;
 }