[PATCH 16/16] drivers, net, intersil: convert request_context.refcount from atomic_t to refcount_t

Kalle Valo kvalo at codeaurora.org
Wed Apr 5 03:33:12 PDT 2017


"Reshetova, Elena" <elena.reshetova at intel.com> writes:

>> Elena Reshetova <elena.reshetova at intel.com> writes:
>> 
>> > refcount_t type and corresponding API should be
>> > used instead of atomic_t when the variable is used as
>> > a reference counter. This allows to avoid accidental
>> > refcounter overflows that might lead to use-after-free
>> > situations.
>> >
>> > Signed-off-by: Elena Reshetova <elena.reshetova at intel.com>
>> > Signed-off-by: Hans Liljestrand <ishkamiel at gmail.com>
>> > Signed-off-by: Kees Cook <keescook at chromium.org>
>> > Signed-off-by: David Windsor <dwindsor at gmail.com>
>> > ---
>> >  drivers/net/wireless/intersil/orinoco/orinoco_usb.c | 15 ++++++++-------
>> >  1 file changed, 8 insertions(+), 7 deletions(-)
>> 
>> The prefix should be "orinoco_usb:", I'll fix that.
>
> Thanks for both! Will you take the patches in?

You mean patches 15 and 16? I didn't even look rest of the patches.

But not sure yet because Dave doesn't seem to like the interface and I
want to follow the discussion first.

-- 
Kalle Valo



More information about the linux-arm-kernel mailing list