added a few null checks

Aaron Boxer · Aaron Boxer · commit 91eb748947a1 · 2011-03-02T14:07:41.000-05:00
diff --git a/src/ServiceStack.Redis/Support/Queue/Implementation/RedisSequentialWorkQueue.cs b/src/ServiceStack.Redis/Support/Queue/Implementation/RedisSequentialWorkQueue.cs
@@ -203,6 +203,9 @@ public void Update(string workItemId, int index, T newWorkItem)
         /// <param name="itemCount"></param>
         private void Pop(string workItemId, int itemCount)
         {
+            if (itemCount <= 0)
+                return;
+
             using (var disposableClient = clientManager.GetDisposableClient<SerializingRedisClient>())
             {
                 var client = disposableClient.Client;
@@ -254,7 +257,10 @@ public bool HarvestZombies()
         /// <returns>true if lock is released, either by this method or by another client; false otherwise</returns>
         public bool TryForceReleaseLock(SerializingRedisClient client, string workItemId)
         {
-            bool rc = false;
+            if (workItemId == null)
+                return false;
+            
+            var rc = false;
 
             var dequeueLockKey = GlobalDequeueLockKey(workItemId);
             // handle possibliity of crashed client still holding the lock
