commit f13042ec7ea6842cbee81590f40b0e5c2e25a9bf
author Marcin Ślusarz <marcin.slusarz@intel.com> Fri Jul 24 18:14:24 2020 +0200
committer Marge Bot <eric+marge@anholt.net> Thu Jul 30 10:41:00 2020 +0000
tree 8ca9b418a6991246e1e4091a4776cd9cfb118539
parent 59bb0ff945f5dfd71647f5b5090fc0a048338158

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>

src/util/u_process.c

diff --git a/src/util/u_process.c b/src/util/u_process.c
index 74c4d10..68fda93 100644
--- a/src/util/u_process.c
+++ b/src/util/u_process.c
@@ -187,6 +187,9 @@
 
     return 0;
 success:
+   if (r == len)
+      return 0;
+
     process_path[r] = '\0';
     return r;