tree d9ee5cf00ae47ef34504131f0be063f581998684
parent f5b9e3b9a1df83ec3a6d219d7c28a1ac5bc0f339
author Michal Srb <msrb@suse.com> 1534244873 +0200
committer Derek Foreman <derek.foreman.samsung@gmail.com> 1534521560 -0500

connection: Prevent pointer overflow from large lengths.

If the remote side sends sufficiently large `length` field, it will
overflow the `p` pointer. Technically it is undefined behavior, in
practice it makes `p < end`, so the length check passes. Attempts to
access the data later causes crashes.

This issue manifests only on 32bit systems, but the behavior is
undefined everywhere.
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Derek Foreman <derek.foreman.samsung@gmail.com>