diff options
| author | Giuseppe Scrivano <gscrivan@redhat.com> | 2020-04-13 13:35:38 +0200 |
|---|---|---|
| committer | Giuseppe Scrivano <gscrivan@redhat.com> | 2020-04-13 13:35:38 +0200 |
| commit | c33371fadbcd933318c34963e9e161209c492da6 (patch) | |
| tree | 3eb4b9964c3e3ef9dad8cf926a3e5e15bb81fb61 /pkg | |
| parent | 291ad7fbd9797340015b71b448c6637669176878 (diff) | |
| download | podman-c33371fadbcd933318c34963e9e161209c492da6.tar.gz podman-c33371fadbcd933318c34963e9e161209c492da6.tar.bz2 podman-c33371fadbcd933318c34963e9e161209c492da6.zip | |
rootless: use snprintf
use directly snprintf instead of strlen+strcpy.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/rootless/rootless_linux.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/pkg/rootless/rootless_linux.c b/pkg/rootless/rootless_linux.c index 6643bfbbf..da52a7217 100644 --- a/pkg/rootless/rootless_linux.c +++ b/pkg/rootless/rootless_linux.c @@ -288,6 +288,7 @@ static void __attribute__((constructor)) init() char *cwd = getcwd (NULL, 0); char uid_fmt[16]; char gid_fmt[16]; + size_t len; if (cwd == NULL) { @@ -295,13 +296,13 @@ static void __attribute__((constructor)) init() _exit (EXIT_FAILURE); } - if (strlen (xdg_runtime_dir) >= PATH_MAX - strlen (suffix)) + len = snprintf (path, PATH_MAX, "%s%s", xdg_runtime_dir, suffix); + if (len >= PATH_MAX) { fprintf (stderr, "invalid value for XDG_RUNTIME_DIR: %s", strerror (ENAMETOOLONG)); exit (EXIT_FAILURE); } - sprintf (path, "%s%s", xdg_runtime_dir, suffix); fd = open (path, O_RDONLY); if (fd < 0) { |