perf config: Modify size factor of snprintf
authorSihyeon Jang <uneedsihyeon@gmail.com>
Sat, 1 Dec 2018 15:46:03 +0000 (00:46 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 17 Dec 2018 17:59:40 +0000 (14:59 -0300)
According to definition of snprintf, it gets size factor including
null('\0') byte.  So '-1' is not neccessary. Also it will be helpful
unfied style with other cases. (eg. builtin-script.c)

Signed-off-by: Sihyeon Jang <uneedsihyeon@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/20181201154603.10093-1-uneedsihyeon@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/config.c

index 3beb4cf44c31def05609a28702ff33d11ca896d6..1ea8f898f1a1c253876ec796f0d8e75eed375bbd 100644 (file)
@@ -815,14 +815,14 @@ int config_error_nonbool(const char *var)
 void set_buildid_dir(const char *dir)
 {
        if (dir)
-               scnprintf(buildid_dir, MAXPATHLEN-1, "%s", dir);
+               scnprintf(buildid_dir, MAXPATHLEN, "%s", dir);
 
        /* default to $HOME/.debug */
        if (buildid_dir[0] == '\0') {
                char *home = getenv("HOME");
 
                if (home) {
-                       snprintf(buildid_dir, MAXPATHLEN-1, "%s/%s",
+                       snprintf(buildid_dir, MAXPATHLEN, "%s/%s",
                                 home, DEBUG_CACHE_DIR);
                } else {
                        strncpy(buildid_dir, DEBUG_CACHE_DIR, MAXPATHLEN-1);