Remove unnecessary strnlen() call 17/166517/2
authorVyacheslav Cherkashin <v.cherkashin@samsung.com>
Wed, 10 Jan 2018 13:45:43 +0000 (16:45 +0300)
committerVyacheslav Cherkashin <v.cherkashin@samsung.com>
Wed, 10 Jan 2018 13:56:10 +0000 (16:56 +0300)
Change-Id: I71b957ba094f45ca3bd70060c1e7add2fe3eb0a1
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>
daemon/sys_stat.c

index 3e01b51d4d60cc7e171aa3361baa9c54b13be7c5..cc6cec082cf867b401755426e5d4ee9da21e4885 100644 (file)
@@ -919,6 +919,7 @@ static int read_mapinfo_section(FILE* fp, mapinfo_section_t *map)
        char buf[LARGE_BUFFER];
        const size_t name_size = sizeof(map->name);
        char *name_buf = map->name;
+       const char *src_name = "[anon]";
 
        line_cnt = smaps_lcnt;
        if ((--line_cnt <= 0) || (line = fgets(buf, sizeof(buf), fp)) == 0)
@@ -929,12 +930,10 @@ static int read_mapinfo_section(FILE* fp, mapinfo_section_t *map)
                return 0;
 
        /* map name fill */
-       if (len < NAME_OFFSET + 1) {
-               strncpy(name_buf, "[anon]", name_size);
-       } else {
-               len = strnlen(buf + NAME_OFFSET, sizeof(buf) - NAME_OFFSET);
-               strncpy(name_buf, buf + NAME_OFFSET, name_size);
-       }
+       if (len >= NAME_OFFSET + 1)
+               src_name = buf + NAME_OFFSET;
+
+       strncpy(name_buf, src_name, name_size);
        name_buf[name_size - 1] = '\0';
 
        while (line_cnt-- && (line = fgets(buf, sizeof(buf), fp))) {