commit 8e6a14c63d6b73cde87cb331439f2a4d19cba5b9
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Jul 14 10:37:25 2019 -0700

    libICE 1.0.10
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit b6aad584c1dc278364c295165512b5f5b98c173e
Author: Olivier Fourdan <ofourdan@redhat.com>
Date:   Thu Apr 11 09:05:15 2019 +0200

    cleanup: Separate variable assignment and test
    
    Assigning and testing a value in a single statement hinders code clarity
    and may confuses static code analyzers.
    
    Separate the assignment and the test for clarity.
    
    Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

commit 772e5b0fdfc9dbd8bec070bd0c4c7eb5565df2ee
Author: Olivier Fourdan <ofourdan@redhat.com>
Date:   Wed Apr 10 11:15:11 2019 +0200

    _IceRead: Avoid possible use-after-free
    
    `_IceRead()` gets called from multiple places which do not expect the
    connection to be freed.
    
    Do not free the connection data in `_IceRead()` to avoid potential
    use-after-free issue in the various callers.
    
    The connection data will be freed eventually in `ProcessWantToClose()`,
    so not freeing it in `_IceRead()` should not introduce an memory leak.
    
    Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

commit 1493beba2aa03bdadeed8b4fa5d424df6e113071
Author: Olivier Fourdan <ofourdan@redhat.com>
Date:   Wed Apr 10 11:01:31 2019 +0200

    IceListenForWellKnownConnections: Fix memleak
    
    The function `_IceTransMakeAllCOTSServerListeners` allocates memory for
    `transConns` which is leaked in case of error.
    
    Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

commit a67a477eefdc93c32fa82da6ff0b4e69dd4c2ccb
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Mar 24 15:29:34 2019 -0700

    IceWritePad: always use zero values for pad bytes
    
    Previously it would just bump the pointer in the buffer leaving
    whatever values were previously there in place.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 2318ace3340009c44e78eab094f159f0e0b4a197
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Mar 24 14:36:10 2019 -0700

    IceOpenConnection: check for malloc failure on connect_to_you too
    
    Fixes: https://gitlab.freedesktop.org/xorg/lib/libice/issues/4
    
    Reported-by: mahendra <mahendra.n@samsung.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit b484311c929a1b64966d89da92fafce7263006e1
Author: Allison Lortie <desrt@desrt.ca>
Date:   Tue Jun 14 16:09:46 2016 -0400

    authutil: support $XDG_RUNTIME_DIR/ICEauthority
    
    If we find that $XDG_RUNTIME_DIR is set (and $ICEAUTHORITY is not), then
    the ICEauthority file is stored in the XDG_RUNTIME_DIR instead of the
    home directory, and without a leading dot.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=49173
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 48ed5e04b5a8ba64dcfeea090cf3a32d3087b749
Author: Allison Lortie <desrt@desrt.ca>
Date:   Tue Jun 14 16:08:21 2016 -0400

    authutil: fix an out-of-bounds access
    
    There is a theoretical edge case where the $HOME environment variable
    could be set to the empty string.  IceAuthFileName() unconditionally
    checks index 1 of this string, which is out of bounds.
    
    Fix that up by rejecting empty strings in the same way as we reject
    NULL.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=49173
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 468b83ec4810b4ea2373182b5801f998f3dcd471
Author: Tobias Stoeckmann <tobias@stoeckmann.org>
Date:   Mon Jul 30 20:50:58 2018 +0200

    Always terminate strncpy results.
    
    The function strncpy does not guarantee to append a terminating
    NUL character to the destination.
    
    This patch merges libSM's way of handling this issue into libICE.
    
    Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 1cc4ae8648590f04557a20c8d88d39cef7fe8119
Author: walter harms <wharms@bfs.de>
Date:   Thu Sep 7 18:52:13 2017 +0200

    iceauth.c: FIX warning: unused variable 'ret' in 'arc4random_buf'
    
    commit ff5e59f32255913bb1cdf51441b98c9107ae165b left ret outside the #if
    causing a gcc warning:
    
     In function 'arc4random_buf':
     iceauth.c:89:13: warning: unused variable 'ret' [-Wunused-variable]
    
    fixed by moving  #if 1 up
    
    Signed-off-by:  Walter Harms <wharms@bfs.de>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit ccbcae7d3409789bf346ca35963264d064f54cba
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Dec 7 19:29:55 2018 -0800

    Update configure.ac bug URL for gitlab migration
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit e72ca90879db149bbee6232dd78a565e630e917d
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Nov 18 21:48:59 2018 -0800

    Update README for gitlab migration
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit dc73ec034c9083b8c7d980e80eb6d4c88bcfaa51
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Nov 10 13:13:45 2018 -0800

    Remove obsolete B16 & B32 tags in struct definitions
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit f66955f7250d7c150dfb97862878acc2222781e5
Author: walter harms <wharms@bfs.de>
Date:   Fri Sep 8 20:03:03 2017 +0200

    make IceProtocolShutdown() more readable
    
    I found IceProtocolShutdown() hard to read only to find that was
    it does it aktually very simple. So i rearranged the code to make
    it more readable.
    
    Signed-off-by: Walter Harms <wharms@bfs.de>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
    [Emil Velikov: whitespace fixes]
    Signed-off-by: Emil Velikov <emil.velikov@collabora.com>

commit 936dcaac07f7db569ed91a34e0a4b5944aac205f
Author: walter harms <wharms@bfs.de>
Date:   Fri Sep 8 19:59:17 2017 +0200

    Drop NULL check prior to free()
    
    free() can handle NULL just fine - remove the check.
    
    Signed-off-by: Walter Harms <wharms@bfs.de>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit 43644931cb9cb5cc92391f6f5431535b9b7a3f24
Author: Eric Engestrom <eric.engestrom@imgtec.com>
Date:   Fri Jul 7 11:23:48 2017 +0100

    Make sure string is never NULL
    
    `error_message` is passed in to strncpy() without any check, which
    doesn't handle NULL itself, so let's make it a valid empty string in
    cases where it was NULL.
    
    Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
    Acked-by: Walter Harms <wharms@bfs.de>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit e8c21056134498c49733f6baf572ffbb051ed886
Author: Eric Engestrom <eric.engestrom@imgtec.com>
Date:   Fri Jul 7 11:23:47 2017 +0100

    Make sure error_message is a free-able string
    
    Similar to the previous commit, assigning a static string would crash
    upon freeing.
    
    Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
    Acked-by: Walter Harms <wharms@bfs.de>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit 7a7844bf5ade915268fe7f9b292908c6cd75f3ba
Author: Eric Engestrom <eric.engestrom@imgtec.com>
Date:   Fri Jul 7 11:23:46 2017 +0100

    Make sure errorStr is a free-able string
    
    If the `errorClass` isn't handled by the switch, `errorStr`'s initial
    value would be a pointer to some static memory with an empty string,
    and freeing it would most likely crash.
    
    Let's set it to NULL instead, as is done in other similar places.
    
    Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
    Acked-by: Walter Harms <wharms@bfs.de>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit 7ef9680caa8c223a09beb637e26fd3471128e6ba
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Sun May 8 09:19:36 2016 +0100

    configure.ac: set TRANS_CLIENT/SERVER
    
    Similar to ICE_t just set the define globally and remove the multiple
    definitions throughout the tree
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Adam Jackson <ajax@redhat.com>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (IRC)

commit ab64a947b5de5b778f31ede9cfce386566023a14
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Sun May 8 09:19:35 2016 +0100

    Kill off local ICE_t definitions
    
    Already defined at global scale in configure.ac
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Adam Jackson <ajax@redhat.com>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (IRC)

commit f4c00d345edf3dad5893b50ff0ae7cd3e0cfd780
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Sun May 8 09:19:34 2016 +0100

    Remove unneeded ^L symbols.
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Adam Jackson <ajax@redhat.com>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (IRC)

commit d603d7d7d989c4ff1094810e9fcf2a29bc00bb0c
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Sun May 8 09:19:33 2016 +0100

    Kill off Time_t macro
    
    Analogous to previous commit, including the megacommit that removed the
    need for it.
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Adam Jackson <ajax@redhat.com>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (IRC)

commit 82250f26fc76d7b7574cfd472646a98e325d944a
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Sun May 8 09:19:32 2016 +0100

    Kill off Strstr macro
    
    Directly use the strstr function as opposed to wrapping it in a macro.
    The latter is no longer needed as of
    
    commit 72e353567f8927996a26e72848d86f692c3f0737
    Author: Kaleb Keithley <kaleb@freedesktop.org>
    Date:   Fri Nov 14 16:48:46 2003 +0000
    
        XFree86 4.3.0.1
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Adam Jackson <ajax@redhat.com>
    Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (IRC)

commit ff5e59f32255913bb1cdf51441b98c9107ae165b
Author: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Date:   Tue Apr 4 19:12:53 2017 +0200

    Use getentropy() if arc4random_buf() is not available
    
    This allows to fix CVE-2017-2626 on Linux platforms without pulling in
    libbsd.
    The libc getentropy() is available since glibc 2.25 but also on OpenBSD.
    For Linux, we need at least a v3.17 kernel. If the recommended
    arc4random_buf() function is not available, emulate it by first trying
    to use getentropy() on a supported glibc and kernel. If the call fails,
    fall back to the current (partly vulnerable) code.
    
    Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 1746abbb1ae1c41ba29c14895c5bd3f1334faef5
Author: Mihail Konev <k.mvc@ya.ru>
Date:   Thu Jan 26 13:52:49 2017 +1000

    autogen: add default patch prefix
    
    Signed-off-by: Mihail Konev <k.mvc@ya.ru>

commit 3aa14db63fefb7634b1bd4370e33ba14c4ea90ae
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Mon Mar 9 12:00:52 2015 +0000

    autogen.sh: use quoted string variables
    
    Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
    fall-outs, when they contain space.
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit d41c57eaa0c1474acf0a6fb271f22106e3070016
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Jan 24 10:32:07 2017 +1000

    autogen.sh: use exec instead of waiting for configure to finish
    
    Syncs the invocation of configure with the one from the server.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit ac4bb20e74e064b219de70e9b54516a921fdb7c3
Author: Tobias Stoeckmann <tobias@stoeckmann.org>
Date:   Tue Nov 22 20:13:29 2016 +0100

    Fix use after free on subsequent calls
    
    The function IceAuthFileName is vulnerable to a use after free. The
    flaw can be triggered by calling the function three times:
    
    - First call succeeds and stores the path in buf, a dynamically
      allocated buffer with size bsize.
    - Second call fails due to out of memory. It frees buf, but keeps
      the old size in bsize.
    - Third call only checks if bsize is large enough. Then it uses
      buf without allocating it again -- the use after free happens.
    
    In order to exploit this, an attacker must change environment variables
    between each call, namely ICEAUTHORITY or HOME. It also takes subsequent
    calls. Due to these limitations, I don't consider this to be of high
    priority.
    
    Reviewed-by: Matthieu Herrb <matthieu@herrb.eu>

commit b1720edc9b9f3e7a05caa3fcd81761e5818ea255
Author: Remko van der Vossen <bugs@yuugen.jp>
Date:   Sun Jul 19 08:34:11 2015 -0700

    Bug 90616 - libICE build fails on array bounds check
    
    https://bugs.freedesktop.org/show_bug.cgi?id=90616
    
    Recent versions of gcc have array bounds checking turned on by default,
    this leads to build failures of libICE. As the _IceVersionCount variable
    in ICElibint.h is not declared const the compiler cannot assume that the
    nested for loop in ProcessConnectionSetup stays within bounds.
    
    The simple fix is of course to change the declarations of _IceVersionCount,
    _IceVersions, and the local variable myVersionCount to const declarations.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 8a511dad53774693ed818d54d7896e1663942b18
Author: Jon TURNEY <jon.turney@dronecode.org.uk>
Date:   Sat Sep 13 17:13:44 2014 +0100

    Include unistd.h for getpid()
    
    Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
    Reviewed-by: David Macek <david.macek.0@gmail.com>

commit fd22b62ae6380ddb00fa4c750f5ce175d2a6e76f
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Sep 14 13:08:17 2014 -0700

    spec: Convert troff \*Q..\*U to DocBook <quote>...</quote>
    
    Reported-by: Jasper St. Pierre <jstpierre@mecheye.net>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>

commit 0dfab4253e26d5c6e5f058126eb5e9f7a7732ae8
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Jun 6 18:28:28 2014 -0700

    libICE 1.0.9
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 9fb6ba03d7183ae06644f8a747fdb99b970d65fc
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Dec 24 09:35:02 2013 -0800

    Delete unused name variable in register.c
    
    Found by cppcheck 1.62:
    [src/register.c:84]: (style) Variable 'name' is assigned a value
     that is never used.
    [src/register.c:182]: (style) Variable 'name' is assigned a value
     that is never used.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit c5060918164168c0a4f737b76e92df3c03356dc6
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Dec 24 09:18:17 2013 -0800

    Free iceConn->connection_string when unwinding after malloc fails
    
    Found by cppcheck 1.62:
    [src/accept.c:113]: (error) Memory leak: iceConn.connection_string
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 80f62c54fbd50a3bbdf9c37258525098c9117830
Author: Matthieu Herrb <matthieu.herrb@laas.fr>
Date:   Thu Aug 29 22:18:14 2013 +0200

    Use arc4random when available to produce the auth cookie.
    
    arc4random() and associated functions can be found in libbsd on
    GNU/Linux systems.
    
    Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 2312ee00402088307e69589c3d12529b5232df66
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 23:01:30 2013 -0700

    Make STORE_STRING cast strlen result to CARD16 when storing in CARD16
    
    Clears a number of clang warnings of the form:
    
    connect.c:328:6: warning: implicit conversion loses integer precision:
       'size_t' (aka 'unsigned long') to 'CARD16' (aka 'unsigned short')
       [-Wconversion]
                STORE_STRING (pData, _IceAuthNames[i]);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ./ICElibint.h:173:19: note: expanded from macro 'STORE_STRING'
        CARD16 _len = strlen (_string); \
               ~~~~   ^~~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 6d6aa84dc6acb2daa3ef7e20942c38a1416bf543
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 22:58:46 2013 -0700

    Cast assignments in IceErrorHeader() macro
    
    Clears many clang warnings about implicit conversions losing
    integer precision, such as from storing ints into CARD8 or CARD16.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 9450bb648ef98efd6f08ea7d14ab8b9ea6e7bb54
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 22:47:41 2013 -0700

    Make write_string call write_counted_string instead of copying it
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 84153eee4db318cd897c464e70cb9f8bc8e469f6
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 22:37:09 2013 -0700

    Mark input arguments to write_string functions as const
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 9a74512ffdc1628f1b87d2191439915c63b9104f
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 22:31:41 2013 -0700

    Stop casting return values from malloc
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 4033226105fa861ab5f0276850afc24c0fa45406
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Aug 8 22:17:27 2013 -0700

    Get rid of casts to (char *) in calls to free()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 10c30ae6a7df1a7c352a1d611a313dffcaa42082
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Jul 20 14:48:33 2013 -0700

    Convert remaining sprintf() call to snprintf()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 0a94633e3d805ca477fba6b7acb673d055a3f43d
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Jul 20 14:46:09 2013 -0700

    Convert strcpy/strcat pairs to snprintf calls
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 0d276835222eeb57de56f56cd9e12611b1d30466
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Jul 20 14:35:11 2013 -0700

    Fix some clang warnings about integer sign/size conversions
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit bb3d8a7767cf260b97c7e019e4fec0ee7d7b65a8
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Jul 20 14:24:04 2013 -0700

    unifdef WORD64
    
    Remove leftover remnants of CRAY support, which still had some
    functions consisting solely of /* NOT IMPLEMENTED YET */ comments.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 9561eca1ad28afee6dcd0aebea3dd20a154ec481
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jul 18 00:36:28 2013 -0700

    Constify filename argument to IceLockAuthFile & IceUnlockAuthFile
    
    Needed to fix const string warnings in iceauth - functions already
    copy provided arguments to temporary local buffer for modifications.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 1b1cf8072b2559e15ac440d5484a29a81d6918c6
Author: Colin Walters <walters@verbum.org>
Date:   Wed Jan 4 17:37:06 2012 -0500

    autogen.sh: Implement GNOME Build API
    
    http://people.gnome.org/~walters/docs/build-api.txt
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit 5e784ca7f37823e62733765371c0b8ed1c58e5aa
Author: Adam Jackson <ajax@redhat.com>
Date:   Tue Jan 15 14:28:48 2013 -0500

    configure: Remove AM_MAINTAINER_MODE
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit e1677ce019219ac164d99f1e04f17caf16fc785a
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Jan 4 19:32:08 2013 -0800

    unifdef -U__UNIXOS2__
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit f166e8bbf16ec214fefdcf02ce81f80442899cb7
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Mar 2 19:59:50 2012 -0800

    libICE 1.0.8
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit c87ecd959d61ae97d8a2bd82efc761bb3e299061
Author: Matt Dew <marcoz@osource.org>
Date:   Mon Jan 9 21:25:18 2012 -0700

    informaltable cleanup
    
    On certain tables, add top and bottom borders to table header
    and a bottom border to the table. This matches what those
    tables in the old pdfs looked like.
    
    the <?dbfo keep-together='always'> prevents tables from
    splitting across pages. Useful for tiny tables.
    
    Converting the colwidth to a floating point, IE, 1* -> 1.0*
    cleans up these build errors:
    WARNING: table-layout="fixed" and column-width unspecified =>
    falling back to proportional-column-width(1)
    
    Signed-off-by: Matt Dew <marcoz@osource.org>

commit 8b4321a37a9fa2ad24bf87afabba6b08dcfded12
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Nov 24 12:42:36 2011 -0800

    Plug minor memory leak in unusual path through ProcessConnectionSetup
    
    Error: Memory leak (CWE 401)
       Memory leak of pointer 'release' allocated with malloc((_len + 1))
            at line 1100 of src/process.c in function 'ProcessConnectionSetup'.
              'release' allocated at line 920 with malloc((_len + 1)).
              release leaks when _i >= hisAuthCount at line 925
                  and i >= _IceAuthCount at line 973
                  and found != 0 at line 998
                  and status != 0 at line 1053
                  and status != 1 at line 1070
                  and accept_setup_now == 0 at line 1082
                  and i >= hisAuthCount at line 1093.
       Memory leak of pointer 'vendor' allocated with malloc((_len + 1))
            at line 1100 of src/process.c in function 'ProcessConnectionSetup'.
              'vendor' allocated at line 919 with malloc((_len + 1)).
              vendor leaks when _i >= hisAuthCount at line 925
                  and i >= _IceAuthCount at line 973
                  and found != 0 at line 998
                  and status != 0 at line 1053
                  and status != 1 at line 1070
                  and accept_setup_now == 0 at line 1082
                  and i >= hisAuthCount at line 1093.
    
    [ This bug was found by the Parfait 0.3.7 bug checking tool.
      For more information see http://labs.oracle.com/projects/parfait/ ]
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit caf9e3393d5d517ea129392d001a2c46a7c1f325
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Nov 19 00:26:29 2011 -0800

    Constify protocol, vendor & release string args to IceRegisterForProtocol*
    
    Needed to resolve gcc -Wwrite-strings warnings in callers.
    These functions only pass the strings to strcmp before calling
    strdup to make their own private copy for storing away.
    
    While fixing the API docs to match, also fix them to match the existing
    function prototypes, where there were several errors before (including
    just plain missing most of the args to IceRegisterForProtocolReply).
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 59c1555dff34804c78c7e7443ad13f27300927eb
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 22:07:34 2011 -0800

    Remove ancient workaround for System V/386 Release 4.2 compiler bug
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb@laas.fr>
    Reviewed-by: walter <wharms@bfs.de>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit bec4e9c9089fa4cc5a50fc513a3544cf6fd59bf1
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 22:06:36 2011 -0800

    Fix gcc -Wwrite-strings warnings in process.c
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit ffa659dd6dd1fb8847bfdcc5e1e2be353b6395b6
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 21:37:48 2011 -0800

    Fix gcc -Wwrite-strings warnings in AuthNames handling
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 902a52ea9d7b6e6f56f7023009859072854a0fc7
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 21:55:17 2011 -0800

    constify arguments to IceGetAuthFileEntry
    
    Needed to clear gcc -Wwrite-strings warnings in callers
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 9ff1f97e46903f8f83363f07cf021989bfa6a9f0
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 21:43:59 2011 -0800

    Fix gcc -Wwrite-strings warnings in _IceError* functions
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit e8a16534a9406d5096d9c0ea515d979c7c15e084
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 9 21:23:33 2011 -0800

    Fix gcc -Wwrite-strings warnings in _IceDefaultErrorHandler
    
    Had to split char *str into two variables, const char *str for the
    string literals just being passed to fprintf etal.  and char *estr
    for use by EXTRACT_STRING for the results of malloc calls that get
    written to and then freed.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 4fbcba4ee19a49c05440861e1278bc97d29048e0
Author: Matt Dew <marcoz@osource.org>
Date:   Tue Oct 4 23:32:02 2011 -0600

    Cleanup IDs and links in doc
    
    1 - fix the capitalization of the ID attributes to match either the
         <title> or <funcdef> string it goes with.
    2 - fix any <linkend>'s that were affected by 1.
    3 - any <function> in the docs that has an actual funcdef,
    will become an olink.
    
    Signed-off-by: Matt Dew <marcoz@osource.org>

commit d4e161e35335df82f412d2d2bb1ef95ff3e88401
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Sep 19 15:34:47 2011 -0400

    As of 1997, X Window System is a trademark of The Open Group.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit fc0fc948d25dfa0f6f445a3b55610ea5b864dd60
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Sep 16 22:09:18 2011 -0700

    Strip trailing whitespace
    
    Performed with: find * -type f | xargs perl -i -p -e 's{[ \t]+$}{}'
    git diff -w & git diff -b show no diffs from this change
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 22601de3f38d035eb874d4bd9f649ddd2b3f0ae2
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 17:20:20 2011 -0400

    specs: remove <productnumber> which is not used by default
    
    This element is not rendered by default on the title. A template
    customization is required to display it.
    X Window System does not have a product number.
    
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 7f330738f1f5862950b3baa35ff08446093bfd4e
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 17:06:00 2011 -0400

    specs: use DocBook suggested markup for Copyrights
    
    Puts the statement on a single line, using commas to separate years
       --> Copyright © 1993, 1994, 1996 X Consortium
    
    As opposed to 3 double-spaced lines.
    
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit d68d2d2031f623a969784e702c97a07b8cd99010
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 16:56:21 2011 -0400

    specs: remove orphan <corpname>
    
    It does not display on the title page and is redundant.
    No visible change on the doc.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 87c934b9b7973a814763670e2dfad0579a8dcc78
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 16:27:09 2011 -0400

    specs: use the &fullrelvers; entity to set X11 release information
    
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 41f04fd8673db5c2d8e587ab2c169a694bd7528f
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 14:13:35 2011 -0400

    ice.xml: version number is 1.1, not 1.0
    
    This is what it was before the conversion from roff to docbook.
    
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 9e2a596b4ff06fc9c70dfcd94506e536e351ee0b
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Sep 11 14:11:28 2011 -0400

    ice.xml: fix orphan affiliation
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 1309d477a061d165345b602e122990eaab71a0d4
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Sep 9 21:21:57 2011 -0400

    ICElib doc: remove empty revision list and specify the doc version number
    
    Revision histories are not used, only 3 docs out of 63 have one.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit e9d57773df26ad9440a49f58941b511e594605a4
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Thu Sep 8 20:00:00 2011 -0400

    docbook.am: embed css styles inside the HTML HEAD element
    
    Rather than referring to the external xorg.css stylesheet, embed the content
    of the file in the html output produced. This is accomplished by using
    version 1.10 of xorg-xhtml.xsl.
    
    This makes the whole html docs tree much more relocatable.
    In addition, it eliminates xorg.css as a runtime file which makes
    xorg-sgml-doctools a build time only package.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit b83cbed755315f6300133f2621cbe99bdc06345a
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Sep 7 10:31:04 2011 -0400

    docbook.am: global maintenance update - entities, images and olinking
    
    Adding support in libX11 for html chunking caused a reorg of docbook.am
    as well as the xorg-sgml-doctools masterdb for olinking.
    The parameter img.src.path is added for pdf images.
    A searchpath to the root builddir is added for local entities, if present.
    
    The docbook.am makefile hides all the details and is identical for
    all 22 modules having DocBook documentation. It is included by a thin
    Makefile.am which requires no docbook knowledge.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit f35b8b8be16f1903beed34fe23fa53d102329342
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Jun 12 17:54:50 2011 -0400

    Install xml versions of specs even if HAVE_XMLTO is false
    
    DocBook/XML input source is also a usefull output format that can be viewed
    with an XML viewer or editor and by some O/S help system.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 4c9cbdb1b40799121456c692d960135d741f13c7
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Jun 5 16:27:36 2011 -0400

    Install target dbs alongside generated documents
    
    This matches a change in xorg-sgml-docs whereby the masterdb will look for
    the target dbs into the same location as the generated documents.
    
    The target dbs are now installed alongside the generated documents.
    Previously they are installed in $prefix/sgml/X11/dbs alongside masterdb which
    has the potential of installing outside the package prefix and cause
    distcheck to fail when user does not have write permission in this package.
    
    Requires XORG_CHECK_SGML_DOCTOOLS(1.8) which was released 2011-06-11

commit 72d668dd2ec4db9b75973ba24f42ab49851dbc6d
Author: Matt Dew <marcoz@osource.org>
Date:   Wed May 25 22:54:51 2011 -0600

    add id attributes to funcsynopsis to allow other docs to olink to them
    
    Signed-off-by: Matt Dew <marcoz@osource.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Gaetan Nadon <memsize@videotron.ca>

commit dd7f2beaa753541aefba499813f25de38cbd17b7
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Apr 19 18:36:58 2011 -0700

    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>

commit 663782989be82e7893c99eaa7cbe98ec25a23c38
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Apr 19 18:19:19 2011 -0700

    Fix reads outside array bounds in error handlers
    
    Error: Buffer overrun
       Read outside array bounds (CWE 125): In call to memcpy(<unknown>, &mOp, 8), (size(&mOp) < (unsigned) 8)
          Array size is 1 bytes
            at line 296 of src/error.c in function '_IceErrorMajorOpcodeDuplicate'.
    Error: Buffer overrun
       Read outside array bounds (CWE 125): In call to memcpy(<unknown>, &maj, 8), (size(&maj) < (unsigned) 8)
          Array size is 1 bytes
            at line 346 of src/error.c in function '_IceErrorBadMajor'.
    
    [ This bug was found by the Parfait 0.3.7 bug checking tool.
      For more information see http://labs.oracle.com/projects/parfait/ ]
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit daf686b7639919289b07a575a0d88efcb91d9732
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Mar 30 20:15:07 2011 -0400

    doc: xorg-sgml-doctools package at version 1.7 is required
    
    This version contains support for external references
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit fadb927532556adb471300c89de29d268aae9102
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Feb 27 15:06:18 2011 -0500

    Documentation: add Docbook external references support
    
    When writing technical documentation, it is often necessary to cross
    reference to other information. When that other information is not in the
    current document, additional support is needed, namely <olink>.
    
    A new feature with version 1.7 of xorg-sgml-doctools adds references to
    other documents within or outside this package.
    
    This patch adds technical support for this feature but does not change
    the content of the documentation as seen by the end user.
    
    Each book or article must generate a database containing the href
    of sections that can be referred to from another document. This database
    is installed in DATAROOTDIR/sgml/X11/dbs. There is a requirement that
    the value of DATAROOTDIR for xorg-sgml-doctools and for the package
    documentation is the same. This forms a virtual document tree.
    
    This database is consulted by other documents while they are being generated
    in order to fulfill the missing information for linking.
    Refer to the xorg-sgml-doctools for further technical information.
    
    Co-authored-by: Matt Dew <marcoz@osource.org>
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 483d32621c06bcba0c7aa7794633b46b151fd5bf
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Feb 25 08:58:54 2011 -0500

    Docbook: change the book id to match the xml file basename
    
    This is required for the up-coming external references support.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 4852c5bb1603fb6d401fe6895d9318555a8d4523
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Feb 2 19:13:54 2011 -0500

    config: splitting ICE and XTRANS compiler options is not required
    
    Simplify configuration by using a single PKG_CHECK_MODULES statement.
    
