diff options
author | Ivan Komissarov <ABBAPOH@gmail.com> | 2019-11-23 11:19:34 +0900 |
---|---|---|
committer | Ivan Komissarov <ABBAPOH@gmail.com> | 2019-11-25 10:49:18 +0000 |
commit | 2af2a33a5d9e938915cf890242b69e6f44f8dafc (patch) | |
tree | 92023b9858ec07c015aa42eaffbb0ce41aba970e | |
parent | 2bbf716c5743ca8d49474cea0560816c42fdbf9f (diff) |
Add a check for the readlink result in processNameByPid()
This fixes release build on Linux
Change-Id: I67fb4b01bf578db625343b50b06c0d1a0457999c
Reviewed-by: Richard Weickelt <richard@weickelt.de>
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
-rw-r--r-- | src/lib/corelib/tools/processutils.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lib/corelib/tools/processutils.cpp b/src/lib/corelib/tools/processutils.cpp index 04a061e0c..b27592f88 100644 --- a/src/lib/corelib/tools/processutils.cpp +++ b/src/lib/corelib/tools/processutils.cpp @@ -92,7 +92,8 @@ QString processNameByPid(qint64 pid) char buf[PATH_MAX]; memset(buf, 0, sizeof(buf)); sprintf(exePath, "/proc/%lld/exe", pid); - readlink(exePath, buf, sizeof(buf)); + if (readlink(exePath, buf, sizeof(buf)) < 0) + return {}; return FileInfo::fileName(QString::fromUtf8(buf)); #elif defined(Q_OS_BSD4) # if defined(Q_OS_NETBSD) |