Fix static analysis issues
authorChanggyu Choi <changyu.choi@samsung.com>
Fri, 30 Dec 2022 01:28:23 +0000 (10:28 +0900)
committerChanggyu Choi <changyu.choi@samsung.com>
Fri, 30 Dec 2022 04:36:28 +0000 (13:36 +0900)
To prevent int to size_t casting, this patch changes the type of
variable which be casted to size_t.

Change-Id: I5a90f83d444124fc6764feea1828b16faf397e51
Signed-off-by: Changgyu Choi <changyu.choi@samsung.com>
src/lib/amd_util.c
src/modules/widget/src/amd_widget.c

index e5bd277c15fbec35bff5c24371960280edea93af..9fd9753a756a82a15eba2d2ad8e2a58e49197f34 100644 (file)
@@ -57,12 +57,12 @@ int _util_save_log(const char *tag, const char *message)
        return _logger_print(__logger, tag, message);
 }
 
-static int __send_message(int sock, const struct iovec *vec, int vec_size,
+static int __send_message(int sock, const struct iovec *vec, size_t vec_size,
                const int *desc, int nr_desc)
 {
        struct msghdr msg = {0,};
        int sndret;
-       int desclen = 0;
+       size_t desclen = 0;
        struct cmsghdr *cmsg = NULL;
        char buff[CMSG_SPACE(sizeof(int) * MAX_NR_OF_DESCRIPTORS)] = {0,};
 
@@ -74,7 +74,7 @@ static int __send_message(int sock, const struct iovec *vec, int vec_size,
                nr_desc = 0;
 
        msg.msg_iov = (struct iovec *)vec;
-       msg.msg_iovlen = (size_t)vec_size;
+       msg.msg_iovlen = vec_size;
 
        /* sending ancillary data */
        if (nr_desc > 0) {
@@ -97,7 +97,7 @@ static int __send_message(int sock, const struct iovec *vec, int vec_size,
                }
 
                /* finished packing updating the corect length */
-               msg.msg_controllen = (size_t)desclen;
+               msg.msg_controllen = desclen;
        } else {
                msg.msg_control = NULL;
                msg.msg_controllen = 0;
index 00d12b023456ff8214eb3820772d34da5fc3941f..6cdded7ded223dfb7d4b35881738b219a037465b 100644 (file)
@@ -913,12 +913,12 @@ static int __validate_widget_caller(amd_request_h req)
        return -1;
 }
 
-static int __send_message(int sock, const struct iovec *vec, int vec_size,
+static int __send_message(int sock, const struct iovec *vec, size_t vec_size,
                const int *desc, int nr_desc)
 {
        struct msghdr msg = {0,};
        int sndret;
-       int desclen = 0;
+       size_t desclen = 0;
        struct cmsghdr *cmsg = NULL;
        char buff[CMSG_SPACE(sizeof(int) * MAX_NR_OF_DESCRIPTORS)] = {0,};
 
@@ -930,7 +930,7 @@ static int __send_message(int sock, const struct iovec *vec, int vec_size,
                nr_desc = 0;
 
        msg.msg_iov = (struct iovec *)vec;
-       msg.msg_iovlen = (size_t)vec_size;
+       msg.msg_iovlen = vec_size;
 
        /* sending ancillary data */
        if (nr_desc > 0) {
@@ -953,7 +953,7 @@ static int __send_message(int sock, const struct iovec *vec, int vec_size,
                }
 
                /* finished packing updating the corect length */
-               msg.msg_controllen = (size_t)desclen;
+               msg.msg_controllen = desclen;
        } else {
                msg.msg_control = NULL;
                msg.msg_controllen = 0;