samples: net: hrpl_border_router: Fix NULL pointer dereference

Null-checking `pkt` before making a call to net_pkt_unref() suggests
that it might be NULL, but the pointer is dereferenced before it's
checked.  Perform the check at the beginning of the function and
dereference and drop its reference unconditionally afterwards.

Coverity-ID: 178789
Signed-off-by: Leandro Pereira <[email protected]>

Author: lpereira

samples/net/rpl_border_router/src/http.c

@@ -1682,25 +1682,23 @@ static void http_received(struct http_ctx *ctx,
 			  const struct sockaddr *dst,
 			  void *user_data)
 {
+	if (!pkt) {
+		NET_DBG("Received NULL packet for unknown reason");
+		return;
+	}
+
 	if (!status) {
 		NET_DBG("Received %d bytes data", net_pkt_appdatalen(pkt));
 
 		if (!strncmp((char *)net_pkt_appdata(pkt), JSON_COAP_PREFIX,
 			     sizeof(JSON_COAP_PREFIX) - 1)) {
 			handle_coap_request(ctx, pkt, user_data);
 		}
-
-
-		if (pkt) {
-			net_pkt_unref(pkt);
-		}
 	} else {
 		NET_ERR("Receive error (%d)", status);
-
-		if (pkt) {
-			net_pkt_unref(pkt);
-		}
 	}
+
+	net_pkt_unref(pkt);
 }
 
 static void http_sent(struct http_ctx *ctx,