diff options
author | Kees Cook <keescook@chromium.org> | 2014-10-20 21:57:08 (GMT) |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2014-10-23 18:02:06 (GMT) |
commit | 598a0df07fc6c4642f9b0497cef1233e41d4c987 (patch) | |
tree | 572f526ff444504d6b3f55db1b9d39ca9a6a03a1 /drivers/net/wireless | |
parent | 34b6d4299923ec9101bbf364440cee36420b3fc0 (diff) | |
download | linux-598a0df07fc6c4642f9b0497cef1233e41d4c987.tar.xz |
rtlwifi: prevent format string usage from leaking
Use "%s" in the workqueue allocation to make sure the rtl_hal_cfg name
can never accidentally leak information via a format string.
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless')
-rw-r--r-- | drivers/net/wireless/rtlwifi/base.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/rtlwifi/base.c b/drivers/net/wireless/rtlwifi/base.c index 58ba718..40b6d1d 100644 --- a/drivers/net/wireless/rtlwifi/base.c +++ b/drivers/net/wireless/rtlwifi/base.c @@ -467,7 +467,7 @@ static void _rtl_init_deferred_work(struct ieee80211_hw *hw) rtl_easy_concurrent_retrytimer_callback, (unsigned long)hw); /* <2> work queue */ rtlpriv->works.hw = hw; - rtlpriv->works.rtl_wq = alloc_workqueue(rtlpriv->cfg->name, 0, 0); + rtlpriv->works.rtl_wq = alloc_workqueue("%s", 0, 0, rtlpriv->cfg->name); INIT_DELAYED_WORK(&rtlpriv->works.watchdog_wq, (void *)rtl_watchdog_wq_callback); INIT_DELAYED_WORK(&rtlpriv->works.ips_nic_off_wq, |