summaryrefslogtreecommitdiff
path: root/src/tool/mapcache.c
diff options
context:
space:
mode:
authorshennetsind <ind@henn.et>2015-01-18 11:37:45 -0200
committershennetsind <ind@henn.et>2015-01-18 11:37:45 -0200
commitd0725afa8363b5f7b0612e087b48013d39339039 (patch)
tree3f521b45f4b0f0d3e7dee31cacfdd78d7be867ab /src/tool/mapcache.c
parenta3c4d675ba19df385be5d1e3966c61de7186da57 (diff)
downloadhercules-d0725afa8363b5f7b0612e087b48013d39339039.tar.gz
hercules-d0725afa8363b5f7b0612e087b48013d39339039.tar.bz2
hercules-d0725afa8363b5f7b0612e087b48013d39339039.tar.xz
hercules-d0725afa8363b5f7b0612e087b48013d39339039.zip
Fixing 38 issues
Addressing out of bounds read/write, pointless null checks on already deferenced variables, dead code. Special Thanks to 4144 and Haruna! Signed-off-by: shennetsind <ind@henn.et>
Diffstat (limited to 'src/tool/mapcache.c')
-rw-r--r--src/tool/mapcache.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/tool/mapcache.c b/src/tool/mapcache.c
index cfd05b0aa..2cc70ebf3 100644
--- a/src/tool/mapcache.c
+++ b/src/tool/mapcache.c
@@ -192,13 +192,13 @@ void process_args(int argc, char *argv[])
for(i = 0; i < argc; i++) {
if(strcmp(argv[i], "-grf") == 0) {
if(++i < argc)
- strcpy(grf_list_file, argv[i]);
+ safestrncpy(grf_list_file, argv[i], sizeof(grf_list_file));
} else if(strcmp(argv[i], "-list") == 0) {
if(++i < argc)
- strcpy(map_list_file, argv[i]);
+ safestrncpy(map_list_file, argv[i], sizeof(map_list_file));
} else if(strcmp(argv[i], "-cache") == 0) {
if(++i < argc)
- strcpy(map_cache_file, argv[i]);
+ safestrncpy(map_cache_file, argv[i], sizeof(map_cache_file));
} else if(strcmp(argv[i], "-rebuild") == 0)
rebuild = 1;
}