net: http: Avoid unnecessary net_pkt error print

jukkar · jukkar · commit bd7d1bda20f7 · 2017-06-02T12:43:52.000+03:00
In some cases the net_pkt can be null when freeing it,
this will print error from net_pkt library. Avoid this by
checking the value of net_pkt before calling net_pkt_unref().

Signed-off-by: Jukka Rissanen &lt;jukka.rissanen@linux.intel.com&gt;
diff --git a/subsys/net/lib/http/http_client.c b/subsys/net/lib/http/http_client.c
@@ -455,7 +455,9 @@ static void recv_cb(struct net_context *net_ctx, struct net_pkt *pkt,
 	}
 
 out:
-	net_pkt_unref(pkt);
+	if (pkt) {
+		net_pkt_unref(pkt);
+	}
 }
 
 static int get_local_addr(struct http_client_ctx *ctx)

Original file line number	Diff line number	Diff line change
`@@ -455,7 +455,9 @@ static void recv_cb(struct net_context net_ctx, struct net_pkt pkt,`
`455`	`455`	`}`
`456`	`456`
`457`	`457`	`out:`
`458`		`- net_pkt_unref(pkt);`
	`458`	`+ if (pkt) {`
	`459`	`+ net_pkt_unref(pkt);`
	`460`	`+ }`
`459`	`461`	`}`
`460`	`462`
`461`	`463`	`static int get_local_addr(struct http_client_ctx *ctx)`