diff options
| author | Slávek Banko <[email protected]> | 2017-03-25 17:33:22 +0100 |
|---|---|---|
| committer | Slávek Banko <[email protected]> | 2017-03-25 17:38:41 +0100 |
| commit | 790f681ae22c49ca313f4238a2c6c807a838f4fa (patch) | |
| tree | fe6160ed704ad71e879a62afb9cbdbabccbdb0b8 /kxkb | |
| parent | 015f35e5f2b63efecba482397a754be3515d4bce (diff) | |
| download | tdebase-790f681ae22c49ca313f4238a2c6c807a838f4fa.tar.gz tdebase-790f681ae22c49ca313f4238a2c6c807a838f4fa.zip | |
Fix memory leak after use XGetAtomName
Signed-off-by: Slávek Banko <[email protected]>
(cherry picked from commit ba2fe631679775cc5e25fe0e7b9769e7bc0e0f68)
Diffstat (limited to 'kxkb')
| -rw-r--r-- | kxkb/kcmmisc.cpp | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/kxkb/kcmmisc.cpp b/kxkb/kcmmisc.cpp index 1687b1c81..358d10450 100644 --- a/kxkb/kcmmisc.cpp +++ b/kxkb/kcmmisc.cpp @@ -274,26 +274,27 @@ int xkb_init() && XkbQueryExtension( dpy, &xkb_opcode, &xkb_event, &xkb_error, &xkb_lmaj, &xkb_lmin ); } - + unsigned int xkb_mask_modifier( XkbDescPtr xkb, const char *name ) - { +{ int i; if( !xkb || !xkb->names ) return 0; - for( i = 0; - i < XkbNumVirtualMods; - i++ ) - { + for( i = 0; i < XkbNumVirtualMods; i++ ) { char* modStr = XGetAtomName( xkb->dpy, xkb->names->vmods[i] ); - if( modStr != NULL && strcmp(name, modStr) == 0 ) - { + if( modStr == NULL ) { + continue; + } + if( strcmp(name, modStr) == 0 ) { unsigned int mask; XkbVirtualModsToReal( xkb, 1 << i, &mask ); + XFree(modStr); return mask; - } } - return 0; + XFree(modStr); } + return 0; +} unsigned int xkb_numlock_mask() { |