Re: [PATCH] ext4: fix OOB read when checking dotdot dir

From: Theodore Ts'o
Date: Wed Mar 19 2025 - 12:46:24 EST


On Wed, Mar 19, 2025 at 09:28:56AM -0700, Linus Torvalds wrote:
> Why would you use 'strcmp()' when you just checked that the length is one?
>
> IOW, if you are talking about "a bit more optimized", please just check
>
> de->name[0] == '.'
>
> after you've checked that the length is 1.

Yes, good point. My suggestion was to optimize htings by avoid
calling strcmp() in most cases, but you're absolutely correct that we
could do even better by *never* calling strcmp().

- Ted