Linux CIFS filesystem development
 help / color / mirror / Atom feed
From: "Pali Rohár" <pali@kernel.org>
To: Namjae Jeon <linkinjeon@kernel.org>, Paulo Alcantara <pc@manguebit.org>
Cc: linux-cifs@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND 00/11] cifs: Various changes
Date: Fri, 21 Aug 2026 23:47:22 +0200	[thread overview]
Message-ID: <20260821214722.q7rkflclyyugdxll@pali> (raw)
In-Reply-To: <20260706184819.22124-1-pali@kernel.org>

Hello, could you look at other of changes in these series?
Steve already merged few of them and others I have rebased (cleanly) on
top of v7.2 release and pushed to my cifs branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pali/linux.git/log/?h=cifs

On Monday 06 July 2026 20:48:08 Pali Rohár wrote:
> This patch series contains various cifs changes which I sent year ago.
> All those patches are rebased on top of the 7.2-rc2 and there is no new
> change in them, just they are rebased for new code base.
> 
> I pushed these changes also into my git branch cifs:
> https://git.kernel.org/pub/scm/linux/kernel/git/pali/linux.git/log/?h=cifs
> 
> 
> Note that patch "cifs: Remove cifs_backup_query_path_info() and replace
> it by cifs_query_path_info()" is removing these lines:
> 
> -			move_cifs_info_to_smb2(&data->fi, fi);
> -			fdi = (FILE_DIRECTORY_INFO *)fi;
> -			si = (FILE_ID_FULL_DIR_INFO *)fi;
> 
> And I have feeling that in this old code (which is being) removed could
> be a buffer overflow as it is casting between incompatible structures.
> I have not spent too much time with it as the code is begin removed.
> But maybe some AI tool could be able to confirm if this is some buffer
> overflow or not.
> 
> 
> Pali Rohár (11):
>   cifs: Fix and improve cifs_is_path_accessible() function
>   cifs: Remove code for querying FILE_INFO_STANDARD via
>     CIFSSMBQPathInfo()
>   cifs: Remove CIFSSMBSetPathInfoFB() fallback function
>   cifs: Remove cifs_backup_query_path_info() and replace it by
>     cifs_query_path_info()
>   cifs: Fix validation of EAs for WSL reparse points
>   cifs: Validate presence of EA $LXMOD for WSL reparse points
>   cifs: Check if server supports EAs before trying to set it for WSL
>   cifs: Extend ->set_EA() callback to allow operate on reparse point
>   cifs: Create native Window socket file compatible also with WSL
>     subsystem
>   cifs: Add support for parsing WSL symlinks in version 1 format
>   cifs: Add support for creating WSL symlinks in version 1 format
> 
>  fs/smb/client/cifsglob.h   |  10 +-
>  fs/smb/client/cifsproto.h  |   1 +
>  fs/smb/client/cifssmb.c    |  79 +++--------
>  fs/smb/client/fs_context.c |  11 +-
>  fs/smb/client/fs_context.h |   3 +-
>  fs/smb/client/inode.c      |  98 +-------------
>  fs/smb/client/link.c       |   3 +-
>  fs/smb/client/reparse.c    | 267 ++++++++++++++++++++++++++++++++-----
>  fs/smb/client/smb1ops.c    |  35 +++--
>  fs/smb/client/smb1proto.h  |   4 +-
>  fs/smb/client/smb2ops.c    |   6 +-
>  fs/smb/client/xattr.c      |   3 +-
>  fs/smb/common/fscc.h       |   5 +-
>  13 files changed, 306 insertions(+), 219 deletions(-)
> 
> -- 
> 2.20.1
> 

  parent reply	other threads:[~2026-08-21 21:47 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-06 18:48 [PATCH RESEND 00/11] cifs: Various changes Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 01/11] cifs: Fix and improve cifs_is_path_accessible() function Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 02/11] cifs: Remove code for querying FILE_INFO_STANDARD via CIFSSMBQPathInfo() Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 03/11] cifs: Remove CIFSSMBSetPathInfoFB() fallback function Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 04/11] cifs: Remove cifs_backup_query_path_info() and replace it by cifs_query_path_info() Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 05/11] cifs: Fix validation of EAs for WSL reparse points Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 06/11] cifs: Validate presence of EA $LXMOD " Pali Rohár
2026-07-06 22:30   ` Steve French
2026-07-06 22:39     ` Pali Rohár
2026-07-06 22:55       ` Steve French
2026-07-06 23:02         ` Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 07/11] cifs: Check if server supports EAs before trying to set it for WSL Pali Rohár
2026-07-06 22:33   ` Steve French
2026-07-06 22:42     ` Pali Rohár
     [not found]       ` <CAH2r5mvDGM7i8EsdPkW681a_AnXOAsHkg38eJqegO7g2gV8qAw@mail.gmail.com>
2026-07-06 22:58         ` Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 08/11] cifs: Extend ->set_EA() callback to allow operate on reparse point Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 09/11] cifs: Create native Window socket file compatible also with WSL subsystem Pali Rohár
2026-08-09 12:09   ` Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 10/11] cifs: Add support for parsing WSL symlinks in version 1 format Pali Rohár
2026-07-06 20:50   ` Pali Rohár
2026-07-06 21:02   ` Pali Rohár
2026-08-09 12:09   ` [PATCH v2] " Pali Rohár
2026-07-06 18:48 ` [PATCH RESEND 11/11] cifs: Add support for creating " Pali Rohár
2026-08-09 12:13   ` [PATCH v2] " Pali Rohár
2026-08-21 21:47 ` Pali Rohár [this message]
2026-08-22 13:28   ` [PATCH RESEND 00/11] cifs: Various changes Namjae Jeon
2026-08-22 13:31     ` Pali Rohár
2026-08-24 22:45       ` Paulo Alcantara
2026-08-25 20:26         ` Pali Rohár
2026-08-25 23:06   ` [PATCH 0/9] " Pali Rohár
2026-08-25 23:06     ` [PATCH 1/9] cifs: Remove code for querying FILE_INFO_STANDARD via CIFSSMBQPathInfo() Pali Rohár
2026-08-25 23:06     ` [PATCH 2/9] cifs: Remove cifs_backup_query_path_info() and replace it by cifs_query_path_info() Pali Rohár
2026-08-25 23:06     ` [PATCH 3/9] cifs: Fix validation of EAs for WSL reparse points Pali Rohár
2026-08-25 23:06     ` [PATCH 4/9] cifs: Validate presence of EA $LXMOD " Pali Rohár
2026-08-25 23:06     ` [PATCH 5/9] cifs: Check if server supports EAs before trying to set it for WSL Pali Rohár
2026-08-25 23:06     ` [PATCH 6/9] cifs: Extend ->set_EA() callback to allow operate on reparse point Pali Rohár
2026-08-25 23:06     ` [PATCH 7/9] cifs: Create native Window socket file compatible also with WSL subsystem Pali Rohár
2026-08-25 23:06     ` [PATCH 8/9] cifs: Add support for parsing WSL symlinks in version 1 format Pali Rohár
2026-08-25 23:06     ` [PATCH 9/9] cifs: Add support for creating " Pali Rohár

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260821214722.q7rkflclyyugdxll@pali \
    --to=pali@kernel.org \
    --cc=linkinjeon@kernel.org \
    --cc=linux-cifs@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pc@manguebit.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox