On Wed, Aug 12, 2020 at 11:54:02AM +0900, Seungil Kang wrote: Thanks for the patch, my comments below.
> When args = "\"\0", "i" will be 0 and args[i-1] is used (line:238) Can you be less ambiguous with the args value? (Perhaps provide a hexdump of it for better understanding) > Because of "i" is an unsigned int type, > the function will access at args[0xFFFFFFFF] > It can make a crash Please, use proper punctuation, I'm lost where is the sentence and what are the logical parts of them. Can you point out to the code that calls this and leads to a crash? > lib/cmdline.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Can you provide a KUnit test module which can check the case? -- With Best Regards, Andy Shevchenko