diff options
| author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-04-19 18:36:58 -0700 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-04-21 16:02:31 -0700 |
| commit | dd7f2beaa753541aefba499813f25de38cbd17b7 (patch) | |
| tree | b226288c93e855d7b3ccfae407a3a0bef98d9c47 | |
| parent | 663782989be82e7893c99eaa7cbe98ec25a23c38 (diff) | |
IceWritePad: Zero fill pad bytes used when buffer is full
Should be rarely hit, since it's only in the case where most of the
message fits in the buffer, but there's not room left for the padding
bytes, but better than sending uninitialized bytes off the stack when
it happens.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
| -rw-r--r-- | include/X11/ICE/ICEmsg.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/X11/ICE/ICEmsg.h b/include/X11/ICE/ICEmsg.h index 6a60f1c..8ca3e5f 100644 --- a/include/X11/ICE/ICEmsg.h +++ b/include/X11/ICE/ICEmsg.h @@ -217,15 +217,15 @@ extern IcePaAuthStatus _IcePaMagicCookie1Proc ( /* * Write pad bytes. Used to force 32 or 64 bit alignment. - * A maxium of 7 pad bytes can be specified. + * A maximum of 7 pad bytes can be specified. */ #define IceWritePad(_iceConn, _bytes) \ { \ if ((_iceConn->outbufptr + (_bytes)) > _iceConn->outbufmax) \ { \ - char _dummy[7]; \ - IceFlush (_iceConn); \ + char _dummy[7] = { 0 }; \ + IceFlush (_iceConn); \ _IceWrite (_iceConn, (unsigned long) (_bytes), _dummy); \ } \ else \ |