commit 34e445674d5fe7b94b702449ac40d4c0a3ca9d12
author Nick Wellnhofer <wellnhofer@aevum.de> Wed May 31 16:48:27 2017 +0200
committer Nick Wellnhofer <wellnhofer@aevum.de> Thu Jun 01 14:31:27 2017 +0200
tree 5842dc7605c11b4053e153063347492f38fae069
parent 474967241cdcce6d3a2fd356079571eee794ec12

Fix undefined behavior in xmlRegExecPushStringInternal

It's stupid, but the behavior of memcpy(NULL, NULL, 0) is undefined.

xmlregexp.c

diff --git a/xmlregexp.c b/xmlregexp.c
index ca3b4f4..7dc6eea 100644
--- a/xmlregexp.c
+++ b/xmlregexp.c
@@ -4089,8 +4089,9 @@
 		    xmlFree(exec->errString);
 		exec->errString = xmlStrdup(value);
 		exec->errState = exec->state;
-		memcpy(exec->errCounts, exec->counts,
-		       exec->comp->nbCounters * sizeof(int));
+                if (exec->comp->nbCounters)
+                    memcpy(exec->errCounts, exec->counts,
+                           exec->comp->nbCounters * sizeof(int));
 	    }
 
 	    /*