Commit 49796d6a authored by Peter Hutterer's avatar Peter Hutterer

Fix format-security warnings

sis_driver.c: In function 'SISErrorLog':
sis_driver.c:432:5: error: format not a string literal and no format arguments [-Werror=format-security]
     xf86DrvMsg(pScrn->scrnIndex, X_ERROR, str);
     ^
sis_driver.c:439:5: error: format not a string literal and no format arguments [-Werror=format-security]
     xf86DrvMsg(pScrn->scrnIndex, X_ERROR, str);
     ^
sis_driver.c: In function 'SiSUpdateXineramaScreenInfo':
sis_driver.c:1917:4: error: format not a string literal and no format arguments [-Werror=format-security]
    xf86DrvMsg(pScrn1->scrnIndex, X_INFO, rectxine);
    ^
sis_driver.c:1924:8: error: format not a string literal and no format arguments [-Werror=format-security]
        xf86DrvMsg(pScrn1->scrnIndex, X_INFO, rectxine);
Signed-off-by: Peter Hutterer's avatarPeter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Alan Coopersmith's avatarAlan Coopersmith <alan.coopersmith@oracle.com>
parent 0a4336ef
...@@ -429,14 +429,14 @@ SISErrorLog(ScrnInfoPtr pScrn, const char *format, ...) ...@@ -429,14 +429,14 @@ SISErrorLog(ScrnInfoPtr pScrn, const char *format, ...)
static const char *str = "**************************************************\n"; static const char *str = "**************************************************\n";
va_start(ap, format); va_start(ap, format);
xf86DrvMsg(pScrn->scrnIndex, X_ERROR, str); xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "%s", str);
xf86DrvMsg(pScrn->scrnIndex, X_ERROR, xf86DrvMsg(pScrn->scrnIndex, X_ERROR,
" ERROR:\n"); " ERROR:\n");
xf86VDrvMsgVerb(pScrn->scrnIndex, X_ERROR, 1, format, ap); xf86VDrvMsgVerb(pScrn->scrnIndex, X_ERROR, 1, format, ap);
va_end(ap); va_end(ap);
xf86DrvMsg(pScrn->scrnIndex, X_ERROR, xf86DrvMsg(pScrn->scrnIndex, X_ERROR,
" END OF MESSAGE\n"); " END OF MESSAGE\n");
xf86DrvMsg(pScrn->scrnIndex, X_ERROR, str); xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "%s", str);
} }
static void static void
...@@ -1914,14 +1914,14 @@ SiSUpdateXineramaScreenInfo(ScrnInfoPtr pScrn1) ...@@ -1914,14 +1914,14 @@ SiSUpdateXineramaScreenInfo(ScrnInfoPtr pScrn1)
if(infochanged && !usenonrect) { if(infochanged && !usenonrect) {
xf86DrvMsg(pScrn1->scrnIndex, X_INFO, xf86DrvMsg(pScrn1->scrnIndex, X_INFO,
"Virtual screen size does not match maximum display modes...\n"); "Virtual screen size does not match maximum display modes...\n");
xf86DrvMsg(pScrn1->scrnIndex, X_INFO, rectxine); xf86DrvMsg(pScrn1->scrnIndex, X_INFO, "%s", rectxine);
} }
} else if(infochanged && usenonrect) { } else if(infochanged && usenonrect) {
usenonrect = FALSE; usenonrect = FALSE;
xf86DrvMsg(pScrn1->scrnIndex, X_INFO, xf86DrvMsg(pScrn1->scrnIndex, X_INFO,
"Only clone modes available for this virtual screen size...\n"); "Only clone modes available for this virtual screen size...\n");
xf86DrvMsg(pScrn1->scrnIndex, X_INFO, rectxine); xf86DrvMsg(pScrn1->scrnIndex, X_INFO, "%s", rectxine);
} }
if(pSiS->maxCRT1_X1) { /* Means we have at least one non-clone mode */ if(pSiS->maxCRT1_X1) { /* Means we have at least one non-clone mode */
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment