]> sourceware.org Git - lvm2.git/commitdiff
cov: dm stats missed terminating null
authorZdenek Kabelac <zkabelac@redhat.com>
Mon, 15 Oct 2018 12:45:16 +0000 (14:45 +0200)
committerZdenek Kabelac <zkabelac@redhat.com>
Mon, 15 Oct 2018 15:49:44 +0000 (17:49 +0200)
Coverity noticed allocating insufficient memory
for the terminating null of the string.

libdm/libdm-stats.c

index e9bfa9de9870399b8801f6e5a890541735db0928..ea8fdf2ef8097d325437e1df6267b14f19a86219 100644 (file)
@@ -1009,7 +1009,7 @@ static int _stats_parse_list(struct dm_stats *dms, const char *resp)
         * dm_task_get_message_response() returns a 'const char *' but
         * since fmemopen also permits "w" it expects a 'char *'.
         */
-       if (!(list_rows = fmemopen((char *)resp, strlen(resp), "r")))
+       if (!(list_rows = fmemopen((char *)resp, strlen(resp) + 1, "r")))
                return_0;
 
        /* begin region table */
@@ -1240,7 +1240,7 @@ static int _stats_parse_region(struct dm_stats *dms, const char *resp,
         * dm_task_get_message_response() returns a 'const char *' but
         * since fmemopen also permits "w" it expects a 'char *'.
         */
-       stats_rows = fmemopen((char *)resp, strlen(resp), "r");
+       stats_rows = fmemopen((char *)resp, strlen(resp) + 1, "r");
        if (!stats_rows)
                goto_bad;
 
This page took 0.04844 seconds and 5 git commands to generate.