src/*/libbpmclient/src/*: fix request message wrong field size

On sending the message, the second parameter is not used, but as we use the same message structure for read/write messages, the server will still check for strict consistency of arguments. Thus, for functions that expect a double argument, enconding a uint32_t will result in an error. The fix is simple, however. We just need to use the passed parameter size to the message.

src//libbpmclient/src/: fix request message wrong field size
On sending the message, the second parameter is not used, but as we use the same message structure for read/write messages, the server will still check for strict consistency of arguments. Thus, for functions that expect a double argument, enconding a uint32_t will result in an error. The fix is simple, however. We just need to use the passed parameter size to the message.
49dd6b08 · Lucas Russo · dc72f794 · 49dd6b08
Commit 49dd6b08 authored Aug 03, 2015 by Lucas Russo
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

bpm_client_rw_param.c src/libs/libbpmclient/src/bpm_client_rw_param.c +5 -2

No files found.
--- a/src/libs/libbpmclient/src/bpm_client_rw_param.c
+++ b/src/libs/libbpmclient/src/bpm_client_rw_param.c
@@ -152,11 +152,14 @@ bpm_client_err_e param_client_read_gen (bpm_client_t *self, char *service,
    assert (service);

    bpm_client_err_e err = BPM_CLIENT_SUCCESS;
-    uint32_t param2 = 0;
    zmsg_t *report;

+    /* Even though we don't use the second parameter, we have the same
+     * message strucuture and the server will check for strict consistency
+     * (number of arguments and size) of all parameters. So, use the size of
+     * the passed parameter here */
    err = param_client_send_gen_rw (self, service, operation, param1,
-            &param2 /* in read mode this value will be ignored */, sizeof (param2));
+            param_out /* in read mode this value will be ignored */, size);
    ASSERT_TEST(err == BPM_CLIENT_SUCCESS, "Could not send message", err_send_msg);
    err = param_client_recv_rw (self, service, &report);
    ASSERT_TEST(err == BPM_CLIENT_SUCCESS, "Could not receive message", err_recv_msg);