diff options
author | Giuseppe Bilotta <giuseppe.bilotta@gmail.com> | 2017-11-09 10:21:20 +0100 |
---|---|---|
committer | Mihai Moldovan <ionic@ionic.de> | 2017-12-07 07:20:55 +0100 |
commit | 4e059d5798f8c0f686c4ef5cec0f6c98ff6d5646 (patch) | |
tree | ed1b3acfedea2910e78a42b8cb567f362acef104 /COPYING | |
parent | c571aa9eda7d69b3de1647a208c685a971c9ad38 (diff) | |
download | nx-libs-4e059d5798f8c0f686c4ef5cec0f6c98ff6d5646.tar.gz nx-libs-4e059d5798f8c0f686c4ef5cec0f6c98ff6d5646.tar.bz2 nx-libs-4e059d5798f8c0f686c4ef5cec0f6c98ff6d5646.zip |
randr: always realloc crtcs and outputs
Backported from X.org:
commit 16381d186e7c791031392ed8afcfd33009854e9e
Author: Giuseppe Bilotta giuseppe.bilotta@gmail.com
Date: Thu Nov 9 10:21:20 2017 +0100
randr: always realloc crtcs and outputs
When the last crtc (resp. output) is destroyed, the rrScrPriv crtcs
(resp. outputs) fields do not get cleared, which can lead to a situation
where the private's numCrtcs (resp. numOutputs) field is zero, but the
associated memory is still allocated. Just checking if numCrtcs (resp.
numOutputs) is zero is thus not a good criteria to determine whetehr to
use a realloc or a malloc.
Since crtcs (resp. outputs) are NULL-initialized anyway, relying on
numCrtcs (resp. numOutputs) is actually unnecessary, because
reallocation of a NULL ptr is equivalent to a malloc anyway.
Therefore, just use realloc() unconditionally, and ensure that the
fields are properly initialized.
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Backported-to-NX-by: Ulrich Sibiller <uli42@gmx.de>
Fixes ArcticaProject/nx-libs#558
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions