diff options
| author | Mateusz Moscicki <m.moscicki2@partner.samsung.com> | 2020-05-06 12:52:36 +0200 |
|---|---|---|
| committer | Mateusz Moscicki <m.moscicki2@partner.samsung.com> | 2020-05-06 15:27:59 +0200 |
| commit | cf5ef5a55bb1d2631dca5b3bcb3c71903938faa8 (patch) | |
| tree | 975ccdfe153f399abd30e68085067924878e23ea | |
| parent | 3a1a00dc1707ed6666bacf82b01d5c8444d5d9f3 (diff) | |
| download | crash-worker-cf5ef5a55bb1d2631dca5b3bcb3c71903938faa8.tar.gz crash-worker-cf5ef5a55bb1d2631dca5b3bcb3c71903938faa8.tar.bz2 crash-worker-cf5ef5a55bb1d2631dca5b3bcb3c71903938faa8.zip | |
Fix Coverity issue
dirname() could return pointer to the local variable
Change-Id: I886df1d2abf73d49f20c11e7ed5eef4595d1c6c5
| -rw-r--r-- | src/crash-manager/so-info.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/crash-manager/so-info.c b/src/crash-manager/so-info.c index d2526d7..6dc61d4 100644 --- a/src/crash-manager/so-info.c +++ b/src/crash-manager/so-info.c @@ -462,10 +462,9 @@ bool correct_file_path(const char *file_path, char *buff, size_t len) strncpy(file_path_copy, file_path, sizeof(file_path_copy) - 1); char *dir_name = dirname(file_path_copy); + char tmp_dir_name[PATH_MAX]; if (!file_exists_in_dir(dir_name, dll_name)) { - char tmp_dir_name[PATH_MAX]; - strncpy(tmp_dir_name, dir_name, sizeof(tmp_dir_name) - 1); dir_name = dirname(tmp_dir_name); |