xdm: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Mon Apr 8 23:15:58 PDT 2013


 xdm/access.c |   54 ++++++++++++++++++++++++++++++++----------------------
 1 file changed, 32 insertions(+), 22 deletions(-)

New commits:
commit b1af740f2824b00fd2e073f847e0873b406bc1ad
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Apr 1 23:37:45 2013 -0700

    Stop leaking address struct on every call to getLocalAddress()
    
    getLocalAddress() checks the haveLocalAddress static variable to see
    if it's already set up an address struct, and if not allocates memory
    to do so and stores a pointer in the localAddress static variable.
    
    Unfortunately, it never then set haveLocalAddress, so every time it was
    called it simply overwrote the previous allocation with a new one.
    
    Now we check to see if the allocation succeeded, and if so, then set
    haveLocalAddress so we don't repeat and waste time & memory on later
    calls.
    
    Reported-by: Ilja Van Sprundel <ivansprundel at ioactive.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list