[PATCH] cifs: Fix printing Status code into dmesg

Steve French smfrench at gmail.com
Tue Jan 21 05:45:18 UTC 2025


On Mon, Jan 20, 2025 at 11:55 AM Pali Rohár <pali at kernel.org> wrote:
>
> 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@kernel.org/t/#u

Both of these are in for-next

> 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 at kernel.org>
> >
> >
> > --
> > Thanks,
> >
> > Steve



-- 
Thanks,

Steve



More information about the samba-technical mailing list