@stevekennedyuk commented on this pull request.


> @@ -74,6 +74,11 @@
        struct meta_layout *m = (struct meta_layout *)malloc(header_len);
        size_t file_offset, tile_size;
 
+       if (m == NULL) {
+               snprintf(log_msg, PATH_MAX - 1, "Failed to allocate memory for 
metatile header\n");

False positive: the format string is a constant literal with no arguments, so 
nothing external can influence it, and `snprintf()` always NUL-terminates when 
the size is non-zero. It's the same `snprintf(log_msg, PATH_MAX - 1, ...)` 
pattern used for the other errors in this function.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/mod_tile/pull/503#discussion_r4261658029
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/mod_tile/pull/503/review/[email protected]>
_______________________________________________
Tile-serving mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/tile-serving

Reply via email to