Skip to content

N
nx-libs
Commit 44e47206 authored by Erkki Seppälä's avatar Erkki Seppälä Committed by Ulrich Sibiller
Browse files
Options

ximcp/imRm: Handle leaking colormap_ret

Fixed memory leak by adding Xfree for colormap_ret Variable "colormap_ret" goes out of scope Reviewed-by: 's avatarAlan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: 's avatarAnder Conselvan de Oliveira <ander.conselvan-de-oliveira@nokia.com> Signed-off-by: 's avatarErkki Seppälä <erkki.seppala@vincit.fi> Signed-off-by: 's avatarAlan Coopersmith <alan.coopersmith@oracle.com> Backported-to-NX-by: 's avatarUlrich Sibiller <uli42@gmx.de>
parent 89dc7ccf
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 0 deletions
nx-X11/lib/X11/imRm.c
View file @ 44e47206
...@@ -2808,6 +2808,8 @@ _XimEncodeLocalPreeditValue( ...@@ -2808,6 +2808,8 @@ _XimEncodeLocalPreeditValue(
ic->core.focus_window, &colormap_ret, ic->core.focus_window, &colormap_ret,
&count, (Atom)p->value))) &count, (Atom)p->value)))
return False; return False;
Xfree(colormap_ret);
} }
return True; return True;
} }
...@@ -2828,6 +2830,8 @@ _XimEncodeLocalStatusValue( ...@@ -2828,6 +2830,8 @@ _XimEncodeLocalStatusValue(
ic->core.focus_window, &colormap_ret, ic->core.focus_window, &colormap_ret,
&count, (Atom)p->value))) &count, (Atom)p->value)))
return False; return False;
Xfree(colormap_ret);
} }
return True; return True;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment