On Tue, Aug 26, 2025 at 10:59:32AM -0400, Jeffrey Altman wrote:
> > The process was running on Ubuntu 24.04, kernel 6.14.0-28-generic, OpenAFS 
> > Client 1.8.13.2-1ubuntu1.
> 
> Ryan observes the behavior on 25.04 and you observe it on 24.04.  Both 
> running 6.14.0 based kernels.  
> 
> The bug was introduced to OpenAFS in commit 
> 0306f3fdac736e15620f5802bdce510d25bb2450 which must have been cherry-picked 
> into the Ubuntu builds.
> 
> struct qstr “quick string” is a counted string not a NUL terminated string.   
> The mistake introduced in the aforementioned commit results in the full path 
> being evaluated as the dentry name instead of just the path component the 
> dentry actually refers to.
> 
> As a result, the d_revalidate() call will always fail, the current dentry 
> will always be invalidated, and a full lookup() will be performed each time a 
> cached dentry is used.
> 
> Marc Dionne should be credited with identifying the mistake.
> 
> This mistake is not present in the AuriStorFS client.

Thanks Jeffrey, I am cc'ing the following people to raise awareness of
this critical bug

Christof Hanke <[email protected]> who appears to have
pulled this commit into the OpenSUSE build for kernel 6.14
compatibility

Jacob Martin <[email protected]> who did the same for Ubuntu:
https://lists.ubuntu.com/archives/plucky-changes/2025-March/014784.html

I filed the following Ubuntu bug report:
https://bugs.launchpad.net/ubuntu/+source/openafs/+bug/2121466

-- 
Ryan C. Underwood, <[email protected]>

Attachment: signature.asc
Description: PGP signature

Reply via email to