diff options
author | Robert Bragg <robert@linux.intel.com> | 2011-12-12 00:30:43 +0000 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-13 18:54:30 -0800 |
commit | 6086f6c1d0e0a1c9e590879acb2319dea0eb6e96 (patch) | |
tree | 022c839a99918494c1b9ecc6ea1727e324f00593 /doc | |
parent | a04a45cb94f7f569e6dd77df93258fa167d0a4ea (diff) |
Xtranssock.c: avoid buffer overrun in SocketReopen
This function was constructing an address from a port string allocating
a buffer according to the size of the string but then later copying
the address according to sizeof(struct sockaddr).
This patch ensures that we allocate a struct sockaddr buffer with enough
space for the port string to be copied into sa_data[] and uses that
combined length to determine how much should be copied at the end of the
function.
This fixes a crash when using xwayland which uses ListenOnOpenFD() that
will call _XSERVTransReopenCOTSServer() with a short port string like
":1".
Signed-off-by: Robert Bragg <robert@linux.intel.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions