From: Furong Xu Date: Thu, 2 Mar 2017 09:00:32 +0000 (+0800) Subject: gpio-button-hotplug: leave platform_device.dev.platform_data untouched X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=5a22df69e421246ba4653d3dd70d25d1aaa5d31c;p=openwrt%2Fstaging%2Fneocturne.git gpio-button-hotplug: leave platform_device.dev.platform_data untouched get platform_data from gpio_keys_button_dev.pdata, and fix a illegal pointer dereference like this: [ 51.143776] gpio-keys-polled gpio-keys-polled: missing poll_interval value [ 51.150852] gpio-keys-polled: probe of gpio-keys-polled failed with error -22 [ 828.159993] gpio-keys-polled gpio-keys-polled: no memory for button data [ 828.166821] gpio-keys-polled: probe of gpio-keys-polled failed with error -12 Signed-off-by: Furong Xu --- diff --git a/package/kernel/gpio-button-hotplug/src/gpio-button-hotplug.c b/package/kernel/gpio-button-hotplug/src/gpio-button-hotplug.c index 2bd3ff1813..1aef23e876 100644 --- a/package/kernel/gpio-button-hotplug/src/gpio-button-hotplug.c +++ b/package/kernel/gpio-button-hotplug/src/gpio-button-hotplug.c @@ -474,7 +474,6 @@ static int gpio_keys_button_probe(struct platform_device *pdev, dev_err(dev, "missing platform data\n"); return -EINVAL; } - pdev->dev.platform_data = pdata; } if (polled && !pdata->poll_interval) { @@ -559,7 +558,7 @@ static int gpio_keys_probe(struct platform_device *pdev) if (ret) return ret; - pdata = pdev->dev.platform_data; + pdata = bdev->pdata; for (i = 0; i < pdata->nbuttons; i++) { struct gpio_keys_button *button = &pdata->buttons[i]; struct gpio_keys_button_data *bdata = &bdev->data[i];