Fixes for QByteArrayMatcher

Copy constructor and assignment operator lose data: pointer to content
and the length of content also need to be copied over.

QByteArrayMatcher::pattern() would return a null byte array if instance
was initialized with c-string.

Changed default constructor to explicitly initialize pattern length to
zero.

The bug in the assignment operator is a regression against 4.4.3.

Task-number: 251958
Reviewed-by: MariusSO
Reviewed-by: paul

1 files changed, 2 insertions, 1 deletions

diff --git a/src/corelib/tools/qbytearraymatcher.cpp b/src/corelib/tools/qbytearraymatcher.cpp
index cd4cf90..211d190 100644
--- a/src/corelib/tools/qbytearraymatcher.cpp
+++ b/src/corelib/tools/qbytearraymatcher.cpp
@@ -120,6 +120,7 @@ QByteArrayMatcher::QByteArrayMatcher()
     : d(0)
 {
     p.p = 0;
+    p.l = 0;
     qMemSet(p.q_skiptable, 0, sizeof(p.q_skiptable));
 }
 
@@ -170,7 +171,7 @@ QByteArrayMatcher::~QByteArrayMatcher()
 QByteArrayMatcher &QByteArrayMatcher::operator=(const QByteArrayMatcher &other)
 {
     q_pattern = other.q_pattern;
-    qMemCopy(p.q_skiptable, other.p.q_skiptable, sizeof(p.q_skiptable));
+    qMemCopy(&p, &other.p, sizeof(p));
     return *this;
 }