Re: [PATCH v1] qnx4_match: do not over run the buffer

2020-11-21 Thread Tong Zhang
Thanks for the clarification! This sounds good to me. I will send a revised patch. Best, - Tong > On Nov 21, 2020, at 4:57 PM, Anders Larsen wrote: > > On Saturday, 2020-11-21 22:47 Tong Zhang wrote: >> >>> On Nov 21, 2020, at 4:40 PM, Anders Larsen wrote: >>> >>> On Friday, 2020-11-20 22:21

Re: [PATCH v1] qnx4_match: do not over run the buffer

2020-11-21 Thread Anders Larsen
On Saturday, 2020-11-21 22:47 Tong Zhang wrote: > > > On Nov 21, 2020, at 4:40 PM, Anders Larsen wrote: > > > > On Friday, 2020-11-20 22:21 Tong Zhang wrote: > >> the di_fname may not terminated by '\0', use strnlen to prevent buffer > >> overrun > >> > >> --- > >> fs/qnx4/namei.c | 2 +- > >> 1

Re: [PATCH v1] qnx4_match: do not over run the buffer

2020-11-21 Thread Tong Zhang
> On Nov 21, 2020, at 4:40 PM, Anders Larsen wrote: > > On Friday, 2020-11-20 22:21 Tong Zhang wrote: >> the di_fname may not terminated by '\0', use strnlen to prevent buffer >> overrun >> >> --- >> fs/qnx4/namei.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/fs

Re: [PATCH v1] qnx4_match: do not over run the buffer

2020-11-21 Thread Anders Larsen
On Friday, 2020-11-20 22:21 Tong Zhang wrote: > the di_fname may not terminated by '\0', use strnlen to prevent buffer > overrun > > --- > fs/qnx4/namei.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/qnx4/namei.c b/fs/qnx4/namei.c > index 8d72221735d7..c0e79094f578

[PATCH v1] qnx4_match: do not over run the buffer

2020-11-20 Thread Tong Zhang
the di_fname may not terminated by '\0', use strnlen to prevent buffer overrun [ 513.248784] qnx4_readdir: bread failed (3718095557) [ 513.250880] == [ 513.251109] BUG: KASAN: use-after-free in strlen+0x1f/0x40 [ 513.251268] Read