diff options
author | Simon Hausmann <simon.hausmann@nokia.com> | 2011-01-28 11:23:08 (GMT) |
---|---|---|
committer | Simon Hausmann <simon.hausmann@nokia.com> | 2011-01-28 12:04:26 (GMT) |
commit | bdf3782b40b0fc2ebfda960be08c90b549cfd970 (patch) | |
tree | 2a3fb241d43ff0769a22a841bc00c41ce39f2b3a /config.tests/unix/shivavg | |
parent | 298b54b4fa99db47d62ec37eacefb4f419b79a69 (diff) | |
download | Qt-bdf3782b40b0fc2ebfda960be08c90b549cfd970.zip Qt-bdf3782b40b0fc2ebfda960be08c90b549cfd970.tar.gz Qt-bdf3782b40b0fc2ebfda960be08c90b549cfd970.tar.bz2 |
Fix potential networking crash due to null-pointer dereference
An internal bug report suggests that we unconditionally dereference the backend
pointer in QNetworkReplyImpl when checking for the synchronity of the
originating request. The dereferencing code was introduced in commit
ad1e82323225e996720136e8b2d669166b8d8441. Unfortunately the report does not
detail where/how the crash happened, but it appears plausible that the backend
pointer became null, and the surrounding code that has extra checks suggests
this, too. In an attempt of defensive programming this patch introduces
the missing check in the reported line 112 as well as in other places
where it seems appropriate.
Reviewed-by: Peter Hartmann
Diffstat (limited to 'config.tests/unix/shivavg')
0 files changed, 0 insertions, 0 deletions