From 27e8395ed0a44338796ea5c4ec48ddd00fa9ec4e Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Mon, 21 Jan 2013 11:26:00 +0000 Subject: [PATCH] Fix crash in wxExecute() introduced by r73406. Don't dereference potentially NULL wxProcess pointer. See #14931. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@73425 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/unix/utilsunx.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/unix/utilsunx.cpp b/src/unix/utilsunx.cpp index eb01ae41c5..07a73e5266 100644 --- a/src/unix/utilsunx.cpp +++ b/src/unix/utilsunx.cpp @@ -556,7 +556,7 @@ long wxExecute(char **argv, int flags, wxProcess *process, // 1. wxPRIORITY_{MIN,DEFAULT,MAX} map to -20, 0 and 19 respectively. // 2. The mapping is monotonously increasing. // 3. The mapping is onto the target range. - int prio = process->GetPriority(); + int prio = process ? process->GetPriority() : 0; if ( prio <= 50 ) prio = (2*prio)/5 - 20; else if ( prio < 55 ) @@ -598,7 +598,7 @@ long wxExecute(char **argv, int flags, wxProcess *process, #endif // !__VMS #if defined(HAVE_SETPRIORITY) - if ( setpriority(PRIO_PROCESS, 0, prio) != 0 ) + if ( prio && setpriority(PRIO_PROCESS, 0, prio) != 0 ) { wxLogSysError(_("Failed to set process priority")); } -- 2.45.2