<feed xmlns='http://www.w3.org/2005/Atom'>
<title>pm24.git/fs/smb/common, branch master</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>https://git.kobert.dev/pm24.git/atom/fs/smb/common?h=master</id>
<link rel='self' href='https://git.kobert.dev/pm24.git/atom/fs/smb/common?h=master'/>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/'/>
<updated>2024-11-25T20:50:32Z</updated>
<entry>
<title>cifs: Add support for parsing WSL-style symlinks</title>
<updated>2024-11-25T20:50:32Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2024-09-28T11:21:24Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=06a7adf318a30bdcfa1222ed6d2640e6bb266d7b'/>
<id>urn:sha1:06a7adf318a30bdcfa1222ed6d2640e6bb266d7b</id>
<content type='text'>
Linux CIFS client currently does not implement readlink() for WSL-style
symlinks. It is only able to detect that file is of WSL-style symlink, but
is not able to read target symlink location.

Add this missing functionality and implement support for parsing content of
WSL-style symlink.

The important note is that symlink target location stored for WSL symlink
reparse point (IO_REPARSE_TAG_LX_SYMLINK) is in UTF-8 encoding instead of
UTF-16 (which is used in whole SMB protocol and also in all other symlink
styles). So for proper locale/cp support it is needed to do conversion from
UTF-8 to local_nls.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: Update comments about some reparse point tags</title>
<updated>2024-09-29T22:28:40Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2024-09-28T21:59:41Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=dd0d7edb866db2b9d23dc50553cbdd3f6a159fba'/>
<id>urn:sha1:dd0d7edb866db2b9d23dc50553cbdd3f6a159fba</id>
<content type='text'>
NFS-style reparse points are recognized only by the Windows NFS server 2012
and new. Windows 8 does not contain Windows NFS server, so these reparse
points are not used on Windows 8.

Reparse points with IO_REPARSE_TAG_AF_UNIX tag were primarily introduced for
native Win32 AF_UNIX sockets and later were re-used by also by WSL:
https://devblogs.microsoft.com/commandline/af_unix-comes-to-windows/
https://devblogs.microsoft.com/commandline/windowswsl-interop-with-af_unix/

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>ksmbd: Correct typos in multiple comments across various files</title>
<updated>2024-09-26T02:33:22Z</updated>
<author>
<name>Shen Lichuan</name>
<email>shenlichuan@vivo.com</email>
</author>
<published>2024-09-25T07:43:23Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=9e676e571d39eb6189bf6d55a9c401ba2dd13410'/>
<id>urn:sha1:9e676e571d39eb6189bf6d55a9c401ba2dd13410</id>
<content type='text'>
Fixed some confusing typos that were currently identified witch codespell,
the details are as follows:

-in the code comments:
fs/smb/common/smb2pdu.h:9: specfication ==&gt; specification
fs/smb/common/smb2pdu.h:494: usally ==&gt; usually
fs/smb/common/smb2pdu.h:1064: Attrubutes ==&gt; Attributes
fs/smb/server/connection.c:28: cleand ==&gt; cleaned
fs/smb/server/ksmbd_netlink.h:216: struture ==&gt; structure
fs/smb/server/oplock.c:799: conains ==&gt; contains
fs/smb/server/oplock.c:1487: containted ==&gt; contained
fs/smb/server/server.c:282: proccessing ==&gt; processing
fs/smb/server/smb_common.c:491: comforms ==&gt; conforms
fs/smb/server/xattr.h:102: ATTRIBUITE ==&gt; ATTRIBUTE

Signed-off-by: Shen Lichuan &lt;shenlichuan@vivo.com&gt;
Acked-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: add comment to STATUS_MCA_OCCURED</title>
<updated>2024-09-15T15:42:44Z</updated>
<author>
<name>ChenXiaoSong</name>
<email>chenxiaosong@kylinos.cn</email>
</author>
<published>2024-08-22T08:21:01Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=e2fcd3fa0351ea2133d1238fcc6a9f140c52d36f'/>
<id>urn:sha1:e2fcd3fa0351ea2133d1238fcc6a9f140c52d36f</id>
<content type='text'>
Explained why the typo was not corrected.

Signed-off-by: ChenXiaoSong &lt;chenxiaosong@kylinos.cn&gt;
Reviewed-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: move SMB2 Status code to common header file</title>
<updated>2024-09-15T15:42:44Z</updated>
<author>
<name>ChenXiaoSong</name>
<email>chenxiaosong@kylinos.cn</email>
</author>
<published>2024-08-22T08:21:00Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=78181a5504a401e421e65d0257a33f904e0e7c29'/>
<id>urn:sha1:78181a5504a401e421e65d0257a33f904e0e7c29</id>
<content type='text'>
There are only 4 different definitions between the client and server:

  - STATUS_SERVER_UNAVAILABLE: from client/smb2status.h
  - STATUS_FILE_NOT_AVAILABLE: from client/smb2status.h
  - STATUS_NO_PREAUTH_INTEGRITY_HASH_OVERLAP: from server/smbstatus.h
  - STATUS_INVALID_LOCK_RANGE: from server/smbstatus.h

Rename client/smb2status.h to common/smb2status.h, and merge the
2 different definitions of server to common header file.

Signed-off-by: ChenXiaoSong &lt;chenxiaosong@kylinos.cn&gt;
Acked-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: move some duplicate definitions to common/smbacl.h</title>
<updated>2024-09-15T15:42:44Z</updated>
<author>
<name>ChenXiaoSong</name>
<email>chenxiaosong@kylinos.cn</email>
</author>
<published>2024-08-22T08:20:59Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=b51174da743b6b7cd87c02e882ebe60dcb99f8bf'/>
<id>urn:sha1:b51174da743b6b7cd87c02e882ebe60dcb99f8bf</id>
<content type='text'>
In order to maintain the code more easily, move duplicate definitions
to new common header file.

Signed-off-by: ChenXiaoSong &lt;chenxiaosong@kylinos.cn&gt;
Acked-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: smb2pdu.h: Use static_assert() to check struct sizes</title>
<updated>2024-08-15T21:06:55Z</updated>
<author>
<name>Gustavo A. R. Silva</name>
<email>gustavoars@kernel.org</email>
</author>
<published>2024-08-08T22:04:04Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=5b4f3af39b6588e8de4444d8e1ccf759b40f9414'/>
<id>urn:sha1:5b4f3af39b6588e8de4444d8e1ccf759b40f9414</id>
<content type='text'>
Commit 9f9bef9bc5c6 ("smb: smb2pdu.h: Avoid -Wflex-array-member-not-at-end
warnings") introduced tagged `struct create_context_hdr`. We want to
ensure that when new members need to be added to the flexible structure,
they are always included within this tagged struct.

So, we use `static_assert()` to ensure that the memory layout for
both the flexible structure and the tagged struct is the same after
any changes.

Acked-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Gustavo A. R. Silva &lt;gustavoars@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>ksmbd: return FILE_DEVICE_DISK instead of super magic</title>
<updated>2024-07-04T04:21:41Z</updated>
<author>
<name>Namjae Jeon</name>
<email>linkinjeon@kernel.org</email>
</author>
<published>2024-06-23T23:39:23Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=25a6e135569b3901452e4863c94560df7c11c492'/>
<id>urn:sha1:25a6e135569b3901452e4863c94560df7c11c492</id>
<content type='text'>
MS-SMB2 specification describes setting -&gt;DeviceType to FILE_DEVICE_DISK
or FILE_DEVICE_CD_ROM. Set FILE_DEVICE_DISK instead of super magic in
FS_DEVICE_INFORMATION. And Set FILE_READ_ONLY_DEVICE for read-only share.

Signed-off-by: Namjae Jeon &lt;linkinjeon@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>fs: smb: common: add missing MODULE_DESCRIPTION() macros</title>
<updated>2024-05-27T05:44:23Z</updated>
<author>
<name>Jeff Johnson</name>
<email>quic_jjohnson@quicinc.com</email>
</author>
<published>2024-05-26T16:52:48Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=9ee267a29309233b9ef8f58ee61e0b1c9b5879e8'/>
<id>urn:sha1:9ee267a29309233b9ef8f58ee61e0b1c9b5879e8</id>
<content type='text'>
Fix the 'make W=1' warnings:
WARNING: modpost: missing MODULE_DESCRIPTION() in fs/smb/common/cifs_arc4.o
WARNING: modpost: missing MODULE_DESCRIPTION() in fs/smb/common/cifs_md4.o

Signed-off-by: Jeff Johnson &lt;quic_jjohnson@quicinc.com&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
<entry>
<title>smb: smb2pdu.h: Avoid -Wflex-array-member-not-at-end warnings</title>
<updated>2024-05-13T21:46:56Z</updated>
<author>
<name>Gustavo A. R. Silva</name>
<email>gustavoars@kernel.org</email>
</author>
<published>2024-04-11T15:35:42Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=9f9bef9bc5c68a6ca9bbffcd29cd188167c29ae1'/>
<id>urn:sha1:9f9bef9bc5c68a6ca9bbffcd29cd188167c29ae1</id>
<content type='text'>
-Wflex-array-member-not-at-end is coming in GCC-14, and we are getting
ready to enable it globally.

So, in order to avoid ending up with a flexible-array member in the
middle of multiple other structs, we use the `__struct_group()` helper
to separate the flexible array from the rest of the members in the
flexible structure, and use the tagged `struct create_context_hdr`
instead of `struct create_context`.

So, with these changes, fix 51 of the following warnings[1]:

fs/smb/client/../common/smb2pdu.h:1225:31: warning: structure containing a flexible array member is not at the end of another structure [-Wflex-array-member-not-at-end]

Link: https://gist.github.com/GustavoARSilva/772526a39be3dd4db39e71497f0a9893 [1]
Link: https://github.com/KSPP/linux/issues/202
Signed-off-by: Gustavo A. R. Silva &lt;gustavoars@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
</content>
</entry>
</feed>
