You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@trafficserver.apache.org by mt...@apache.org on 2010/05/24 06:41:06 UTC

svn commit: r947537 - /trafficserver/traffic/trunk/iocore/cache/CacheRead.cc

Author: mturk
Date: Mon May 24 04:41:06 2010
New Revision: 947537

URL: http://svn.apache.org/viewvc?rev=947537&view=rev
Log:
TS-368: Fix possible null pointer dereference

Modified:
    trafficserver/traffic/trunk/iocore/cache/CacheRead.cc

Modified: trafficserver/traffic/trunk/iocore/cache/CacheRead.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/iocore/cache/CacheRead.cc?rev=947537&r1=947536&r2=947537&view=diff
==============================================================================
--- trafficserver/traffic/trunk/iocore/cache/CacheRead.cc (original)
+++ trafficserver/traffic/trunk/iocore/cache/CacheRead.cc Mon May 24 04:41:06 2010
@@ -56,12 +56,12 @@ Cache::open_read(Continuation * cont, Ca
       c->frag_type = type;
       c->od = od;
     }
+    if (!c)
+      goto Lmiss;
     if (!lock) {
       CONT_SCHED_LOCK_RETRY(c);
       return &c->_action;
     }
-    if (!c)
-      goto Lmiss;
     if (c->od)
       goto Lwriter;
     c->dir = result;