diff options
| author | Peter Hutterer <peter.hutterer@who-t.net> | 2013-05-09 11:06:12 +1000 |
|---|---|---|
| committer | Peter Hutterer <peter.hutterer@who-t.net> | 2013-05-09 11:11:04 +1000 |
| commit | eb50f8023e9ce11c3f0249ef9ddf398a1d983c84 (patch) | |
| tree | ba3f2fd6708d98a0bf44162ffbf7ebc079af99a4 | |
| parent | 7cc0ebf0858ebb505d933839f25bb269a8cdb764 (diff) | |
Plug memory leak for option_string
xf86SetStrOption() returns a strdup'd string that must be freed after use.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | src/synaptics.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/synaptics.c b/src/synaptics.c index 2de05d4..d4b21a9 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -513,12 +513,15 @@ set_softbutton_areas_option(InputInfoPtr pInfo) memcpy(pars->softbutton_areas[0], values, 4 * sizeof(int)); memcpy(pars->softbutton_areas[1], values + 4, 4 * sizeof(int)); + free(option_string); + return; fail: xf86IDrvMsg(pInfo, X_ERROR, "invalid SoftButtonAreas value '%s', keeping defaults\n", option_string); + free(option_string); } static void |