r14003: Clarify code that lead to Coverity report #13.

Not a bug, but better to remove false positives.
Jeremy.
(This used to be commit f9a75d76546bc4618736f0d48646e77d7572db25)

1 files changed, 15 insertions, 12 deletions

diff --git a/source3/printing/print_generic.c b/source3/printing/print_generic.c
index 18fca678600..cb1c951ff74 100644
--- a/source3/printing/print_generic.c
+++ b/source3/printing/print_generic.c
@@ -193,21 +193,24 @@ static int generic_queue_get(const char *printer_name,
 	/* turn the lpq output into a series of job structures */
 	qcount = 0;
 	ZERO_STRUCTP(status);
-	if (numlines)
+	if (numlines) {
 		queue = SMB_MALLOC_ARRAY(print_queue_struct, numlines+1);
-
-	if (queue) {
-		memset(queue, '\0', sizeof(print_queue_struct)*(numlines+1));
-		for (i=0; i<numlines; i++) {
-			/* parse the line */
-			if (parse_lpq_entry(printing_type,qlines[i],
-					    &queue[qcount],status,qcount==0)) {
-				qcount++;
-			}
-		}		
+		if (!queue) {
+			file_lines_free(qlines);
+			*q = NULL;
+			return 0;
+		}
 	}
-	file_lines_free(qlines);
 
+	memset(queue, '\0', sizeof(print_queue_struct)*(numlines+1));
+	for (i=0; i<numlines; i++) {
+		/* parse the line */
+		if (parse_lpq_entry(printing_type,qlines[i],
+				    &queue[qcount],status,qcount==0)) {
+			qcount++;
+		}
+	}		
+	file_lines_free(qlines);
         *q = queue;
 	return qcount;
 }