Re: [PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-08 Thread Saravana Kannan
On 08/07/2017 11:56 PM, Pavan Kondeti wrote: Hi Santosh, On Mon, Aug 7, 2017 at 6:36 PM, Santosh Mardi wrote: store_freq function of devfreq userspace governor executes further, even if error is returned from sscanf, this will result in setting up wrong frequency value. Add proper error check

Re: [PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-07 Thread Pavan Kondeti
Hi Santosh, On Mon, Aug 7, 2017 at 6:36 PM, Santosh Mardi wrote: > store_freq function of devfreq userspace governor > executes further, even if error is returned from sscanf, > this will result in setting up wrong frequency value. > > Add proper error check to bail out if any error is returned.

Re: [PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-07 Thread Chanwoo Choi
Hi, On 2017년 08월 07일 22:06, Santosh Mardi wrote: > store_freq function of devfreq userspace governor > executes further, even if error is returned from sscanf, > this will result in setting up wrong frequency value. > > Add proper error check to bail out if any error is returned. > > Signed-off-

RE: [PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-07 Thread MyungJoo Ham
> store_freq function of devfreq userspace governor > executes further, even if error is returned from sscanf, > this will result in setting up wrong frequency value. > > Add proper error check to bail out if any error is returned. > > Signed-off-by: Santosh Mardi Acked-by: MyungJoo Ham > ---

[PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-07 Thread Santosh Mardi
store_freq function of devfreq userspace governor executes further, even if error is returned from sscanf, this will result in setting up wrong frequency value. Add proper error check to bail out if any error is returned. Signed-off-by: Santosh Mardi --- drivers/devfreq/governor_userspace.c | 5

[PATCH] devfreq: add error check for sscanf in userspace governor

2017-08-07 Thread Santosh Mardi
What this patch does: Current implementation of store_freq function in devfreq userspace governor executes further, even if error is returned from sscanf. This will result in setting up wrong frequency value. This patch adds proper error check to bail out if