Coverity #992: Prevent a NULL chase.

07ecf495 · Adam Jackson · c6b3b335 · 07ecf495 · 07ecf495
Commit 07ecf495 authored Apr 03, 2006 by Adam Jackson 💣
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

ChangeLog ChangeLog +5 -0

Xext/sync.c Xext/sync.c +2 -1

No files found.
--- a/ChangeLog
+++ b/ChangeLog
+2006-04-02  Adam Jackson  <ajax@freedesktop.org>
+
+	* Xext/sync.c:
+	Coverity #992: Prevent a NULL chase.
+
 2006-04-02  Adam Jackson  <ajax@freedesktop.org>

 	* configure.ac:

--- a/Xext/sync.c
+++ b/Xext/sync.c
@@ -1112,7 +1112,7 @@ SyncComputeBracketValues(pCounter, startOver)
 {
    SyncTriggerList *pCur;
    SyncTrigger *pTrigger;
-    SysCounterInfo *psci = pCounter->pSysCounterInfo;
+    SysCounterInfo *psci;
    CARD64 *pnewgtval = NULL;
    CARD64 *pnewltval = NULL;
    SyncCounterType ct;
@@ -1120,6 +1120,7 @@ SyncComputeBracketValues(pCounter, startOver)
    if (!pCounter)
 	return;

+    psci = pCounter->pSysCounterInfo;
    ct = pCounter->pSysCounterInfo->counterType;
    if (ct == XSyncCounterNeverChanges)
 	return;