Re: [PATCH][next] bpf: fix swapped arguments in calls to check_buffer_access

2020-07-28 Thread Yonghong Song
On 7/28/20 3:43 AM, Daniel Borkmann wrote: On 7/27/20 11:39 PM, Yonghong Song wrote: On 7/27/20 10:54 AM, Colin King wrote: From: Colin Ian King There are a couple of arguments of the boolean flag zero_size_allowed and the char pointer buf_info when calling to function check_buffer_access

Re: [PATCH][next] bpf: fix swapped arguments in calls to check_buffer_access

2020-07-28 Thread Colin Ian King
On 28/07/2020 11:43, Daniel Borkmann wrote: > On 7/27/20 11:39 PM, Yonghong Song wrote: >> On 7/27/20 10:54 AM, Colin King wrote: >>> From: Colin Ian King >>> >>> There are a couple of arguments of the boolean flag zero_size_allowed >>> and the char pointer buf_info when calling to function >>> ch

Re: [PATCH][next] bpf: fix swapped arguments in calls to check_buffer_access

2020-07-28 Thread Daniel Borkmann
On 7/27/20 11:39 PM, Yonghong Song wrote: On 7/27/20 10:54 AM, Colin King wrote: From: Colin Ian King There are a couple of arguments of the boolean flag zero_size_allowed and the char pointer buf_info when calling to function check_buffer_access that are swapped by mistake. Fix these by swapp

Re: [PATCH][next] bpf: fix swapped arguments in calls to check_buffer_access

2020-07-27 Thread Yonghong Song
On 7/27/20 10:54 AM, Colin King wrote: From: Colin Ian King There are a couple of arguments of the boolean flag zero_size_allowed and the char pointer buf_info when calling to function check_buffer_access that are swapped by mistake. Fix these by swapping them to correct the argument orderin

[PATCH][next] bpf: fix swapped arguments in calls to check_buffer_access

2020-07-27 Thread Colin King
From: Colin Ian King There are a couple of arguments of the boolean flag zero_size_allowed and the char pointer buf_info when calling to function check_buffer_access that are swapped by mistake. Fix these by swapping them to correct the argument ordering. Addresses-Coverity: ("Array compared to