flow:http:server: Fix wrong comparation

Flavio Ceolin · Flavio Ceolin · commit 84ecbbfe1730 · 2016-03-30T11:59:20.000-03:00
sol_http_param_add() returns a boolean false in case of error. It was
comparing with an integer &lt; 0.

Signed-off-by: Flavio Ceolin &lt;flavio.ceolin@intel.com&gt;
diff --git a/src/modules/flow/http-server/http-server.c b/src/modules/flow/http-server/http-server.c
@@ -296,10 +296,10 @@ common_response_cb(void *data, struct sol_http_request *request)
             "Could not serve request: %s", sol_util_strerrora(-r));
 
         sol_http_params_clear(&response.param);
-        r = sol_http_param_add(&response.param, SOL_HTTP_REQUEST_PARAM_HEADER(
-            HTTP_HEADER_CONTENT_TYPE, HTTP_HEADER_CONTENT_TYPE_TEXT));
-        if (r < 0)
+        if (!sol_http_param_add(&response.param, SOL_HTTP_REQUEST_PARAM_HEADER(
+            HTTP_HEADER_CONTENT_TYPE, HTTP_HEADER_CONTENT_TYPE_TEXT))) {
             SOL_WRN("could not set response content-type: text/plain: %s", sol_util_strerrora(-r));
+        }
 
         /* response_code was set before goto, so use as-is */
         sol_http_server_send_response(request, &response);
