]> git.saurik.com Git - wxWidgets.git/commitdiff
Fix crash in wxExecute() introduced by r73406.
authorVadim Zeitlin <vadim@wxwidgets.org>
Mon, 21 Jan 2013 11:26:00 +0000 (11:26 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Mon, 21 Jan 2013 11:26:00 +0000 (11:26 +0000)
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

index eb01ae41c535d720a755536bcaf6c6ddb800f83a..07a73e52662533ff683f00dd5d382fbe5fecc6d8 100644 (file)
@@ -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"));
         }