Changeset 91789 in vbox for trunk/src/VBox/HostDrivers/Support/solaris

Timestamp:

Oct 17, 2021 6:16:11 PM (3 years ago)

Author:

vboxsync

Message:

SUPDrv,IPRT,VBoxGuest: Don't export ellipsis functions from SUPDrv, as that makes switching back to the kernel stack unsafe. Exports has changes (added+removed), but not bumping major IOC version as that was done a few hours ago already and it's Sunday. bugref:10124

File:

• trunk/src/VBox/HostDrivers/Support/solaris/SUPDrv-solaris.c (modified) (1 diff)

trunk/src/VBox/HostDrivers/Support/solaris/SUPDrv-solaris.c

@@ -1303 +1303 @@
 
 
-RTDECL(int) SUPR0Printf(const char *pszFormat, ...)
-{
-    va_list     args;
-    char        szMsg[512];
-
+RTDECL(int) SUPR0PrintfV(const char *pszFormat, va_list va)
+{
     /* cmn_err() acquires adaptive mutexes. Not preemption safe, see @bugref{6657}. */
-    if (!RTThreadPreemptIsEnabled(NIL_RTTHREAD))
-        return 0;
-
-    va_start(args, pszFormat);
-    RTStrPrintfV(szMsg, sizeof(szMsg) - 1, pszFormat, args);
-    va_end(args);
-
-    szMsg[sizeof(szMsg) - 1] = '\0';
-    cmn_err(CE_CONT, "%s", szMsg);
+    if (RTThreadPreemptIsEnabled(NIL_RTTHREAD))
+    {
+        char szMsg[512];
+        RTStrPrintfV(szMsg, sizeof(szMsg) - 1, pszFormat, va);
+        szMsg[sizeof(szMsg) - 1] = '\0';
+
+        cmn_err(CE_CONT, "%s", szMsg);
+    }
     return 0;
 }