Re: [PATCH v6 7/9] soc: qcom: wcnss_ctrl: Avoid string overflow

2018-09-03 Thread Bjorn Andersson
On Wed 29 Aug 00:57 PDT 2018, Niklas Cassel wrote: > 'chinfo.name' is used as a NUL-terminated string, but using strncpy() with > the length equal to the buffer size may result in lack of the termination: > > drivers//soc/qcom/wcnss_ctrl.c: In function 'qcom_wcnss_open_channel': >

Re: [PATCH v6 7/9] soc: qcom: wcnss_ctrl: Avoid string overflow

2018-09-03 Thread Bjorn Andersson
On Wed 29 Aug 00:57 PDT 2018, Niklas Cassel wrote: > 'chinfo.name' is used as a NUL-terminated string, but using strncpy() with > the length equal to the buffer size may result in lack of the termination: > > drivers//soc/qcom/wcnss_ctrl.c: In function 'qcom_wcnss_open_channel': >

[PATCH v6 7/9] soc: qcom: wcnss_ctrl: Avoid string overflow

2018-08-29 Thread Niklas Cassel
'chinfo.name' is used as a NUL-terminated string, but using strncpy() with the length equal to the buffer size may result in lack of the termination: drivers//soc/qcom/wcnss_ctrl.c: In function 'qcom_wcnss_open_channel': drivers//soc/qcom/wcnss_ctrl.c:284:2: warning: 'strncpy' specified bound 32

[PATCH v6 7/9] soc: qcom: wcnss_ctrl: Avoid string overflow

2018-08-29 Thread Niklas Cassel
'chinfo.name' is used as a NUL-terminated string, but using strncpy() with the length equal to the buffer size may result in lack of the termination: drivers//soc/qcom/wcnss_ctrl.c: In function 'qcom_wcnss_open_channel': drivers//soc/qcom/wcnss_ctrl.c:284:2: warning: 'strncpy' specified bound 32