QXmlStream: fix incorrect use of realloc()

One must never store the result of realloc() back into the first
realloc() argument, because, if relloc() fails, one still has to
free() the old pointer, but one has just overwritten its value with
nullptr, leaking memory.

To fix, first check, and then assign.

Found in manual review of realloc() uses.

Amends the start of the public history.

Pick-to: 6.10 6.9 6.8 6.5
Change-Id: Id6d369116d4e01fb9cb1d6db3627b9956d4b6201
Reviewed-by: Thiago Macieira <[email protected]>

0 files changed, 0 insertions, 0 deletions