fix(pool): ensure workItem is not null before accessing properties

mbroadst · mbroadst · commit 2143963ae686 · 2017-12-01T13:43:28.000-05:00
diff --git a/lib/connection/pool.js b/lib/connection/pool.js
@@ -449,7 +449,7 @@ function messageHandler(self) {
 
     // Reset the connection timeout if we modified it for
     // this operation
-    if (workItem.socketTimeout) {
+    if (workItem && workItem.socketTimeout) {
       connection.resetSocketTimeout();
     }
 
@@ -536,7 +536,7 @@ function messageHandler(self) {
       }
 
       // Time to dispatch the message if we have a callback
-      if (!workItem.immediateRelease) {
+      if (workItem && !workItem.immediateRelease) {
         try {
           // Parse the message according to the provided options
           message.parse(workItem);

Original file line number	Diff line number	Diff line change
`@@ -449,7 +449,7 @@ function messageHandler(self) {`
`449`	`449`
`450`	`450`	`// Reset the connection timeout if we modified it for`
`451`	`451`	`// this operation`
`452`		`- if (workItem.socketTimeout) {`
	`452`	`+ if (workItem && workItem.socketTimeout) {`
`453`	`453`	`connection.resetSocketTimeout();`
`454`	`454`	`}`
`455`	`455`
`@@ -536,7 +536,7 @@ function messageHandler(self) {`
`536`	`536`	`}`
`537`	`537`
`538`	`538`	`// Time to dispatch the message if we have a callback`
`539`		`- if (!workItem.immediateRelease) {`
	`539`	`+ if (workItem && !workItem.immediateRelease) {`
`540`	`540`	`try {`
`541`	`541`	`// Parse the message according to the provided options`
`542`	`542`	`message.parse(workItem);`