diff options
| author | Marcin Ślusarz <marcin.slusarz@intel.com> | 2020-07-24 18:14:24 +0200 |
|---|---|---|
| committer | Eric Engestrom <eric@engestrom.ch> | 2020-08-05 22:23:21 +0200 |
| commit | f2de23e7855cd5346acd3ea32c987ea88e5545db (patch) | |
| tree | a774c81e18e24c254ab3a606f52c82c0dd978f47 /src | |
| parent | 75e81ff0d7720d3167039593e6af8391d776b25c (diff) | |
util: fix possible buffer overflow in util_get_process_exec_path
Found by Coverity.
Signed-off-by: Marcin Ślusarz <marcin.slusarz@intel.com>
Fixes: f8f1413070a ("util/u_process: add util_get_process_exec_path")
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6067>
(cherry picked from commit f13042ec7ea6842cbee81590f40b0e5c2e25a9bf)
Diffstat (limited to 'src')
| -rw-r--r-- | src/util/u_process.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/util/u_process.c b/src/util/u_process.c index 74c4d10192e..68fda93a53b 100644 --- a/src/util/u_process.c +++ b/src/util/u_process.c @@ -187,6 +187,9 @@ util_get_process_exec_path(char* process_path, size_t len) return 0; success: + if (r == len) + return 0; + process_path[r] = '\0'; return r; |