Just to note that I have sent this patch in series with "cifs: Add missing NT_STATUS_* codes from nterr.h to nterr.c" patch which is adding also NT_STATUS_STOPPED_ON_SYMLINK (mentioned in commit message): https://lore.kernel.org/linux-cifs/20241227173709.22892-1-pali@xxxxxxxxxx/t/#u On Sunday 19 January 2025 19:48:39 Steve French wrote: > Any thoughts on the attached patch (which is tentatively in > cifs-2.6.git for-next)? > > NT Status code is 32-bit number, so for comparing two NT Status codes is > needed to check all 32 bits, and not just low 24 bits. > > Before this change kernel printed message: > "Status code returned 0x8000002d NT_STATUS_NOT_COMMITTED" > > It was incorrect as because NT_STATUS_NOT_COMMITTED is defined as > 0xC000002d and 0x8000002d has defined name NT_STATUS_STOPPED_ON_SYMLINK. > > With this change kernel prints message: > "Status code returned 0x8000002d NT_STATUS_STOPPED_ON_SYMLINK" > > Signed-off-by: Pali Rohár <pali@xxxxxxxxxx> > > > -- > Thanks, > > Steve
