Don't call the lock/unlock functions if they are NULL. They can still be

NULL without violating protocol.

diff --git a/lib/share.c b/lib/share.c
index d6cb7cfdc17608ab1f8f52fb311969a62b737a26..ff1aef2e597c91dcca2d7737597064a3c0de55b7 100644 (file)
--- a/lib/share.c
+++ b/lib/share.c
@@ -186,7 +186,8 @@ Curl_share_lock(struct SessionHandle *data, curl_lock_data type,
     return CURLSHE_INVALID;
 
   if(share->specifier & (1<<type)) {
-    share->lockfunc(data, type, accesstype, share->clientdata);
+    if(share->lockfunc) /* only call this if set! */
+      share->lockfunc(data, type, accesstype, share->clientdata);
   }
   /* else if we don't share this, pretend successful lock */
 
@@ -202,7 +203,8 @@ Curl_share_unlock(struct SessionHandle *data, curl_lock_data type)
     return CURLSHE_INVALID;
 
   if(share->specifier & (1<<type)) {
-    share->unlockfunc (data, type, share->clientdata);
+    if(share->unlockfunc) /* only call this if set! */
+      share->unlockfunc (data, type, share->clientdata);
   }
 
   return CURLSHE_OK;