Re: [PATCH 1/2] man/man7/landlock.7: Adapt compatibility in code example to ABI v8

Alejandro Colomar <[email protected]>
Newsgroups org.kernel.vger.linux-man
Message-ID <agW3SQwVtbF6_kJL@devuan>
Hi Günther!

On 2026-05-14T09:04:16+0200, Günther Noack wrote:
> * Bring the code example up to speed with Landlock ABI v8
> * Use the correct ABI level in the MIN() macro
> 
> Signed-off-by: Günther Noack <[email protected]>

Thanks!  I've applied the patch.


Have a lovely day!
Alex

> ---
>  man/man7/landlock.7 | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/man/man7/landlock.7 b/man/man7/landlock.7
> index 06910ccab5b1..0e3a11489af2 100644
> --- a/man/man7/landlock.7
> +++ b/man/man7/landlock.7
> @@ -583,6 +583,9 @@ __u64 landlock_fs_access_rights[] = {
>      (LANDLOCK_ACCESS_FS_TRUNCATE  << 1) \- 1,  /* v3: add "truncate"  */
>      (LANDLOCK_ACCESS_FS_TRUNCATE  << 1) \- 1,  /* v4: TCP support     */
>      (LANDLOCK_ACCESS_FS_IOCTL_DEV << 1) \- 1,  /* v5: add "ioctl_dev" */
> +    (LANDLOCK_ACCESS_FS_IOCTL_DEV << 1) \- 1,  /* v6: same            */
> +    (LANDLOCK_ACCESS_FS_IOCTL_DEV << 1) \- 1,  /* v7: same            */
> +    (LANDLOCK_ACCESS_FS_IOCTL_DEV << 1) \- 1,  /* v8: same            */
>  };
>  \&
>  int abi = landlock_create_ruleset(NULL, 0,
> @@ -595,7 +598,7 @@ if (abi == \-1) {
>      perror("Unable to use Landlock");
>      return;  /* Graceful fallback: Do nothing.  */
>  }
> -abi = MIN(abi, 3);
> +abi = MIN(abi, 8);
>  \&
>  /* Only use the available rights in the ruleset.  */
>  attr.handled_access_fs &= landlock_fs_access_rights[abi \- 1];
> -- 
> 2.54.0
> 

-- 
<https://www.alejandro-colomar.es>
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEES7Jt9u9GbmlWADAi64mZXMKQwqkFAmoFt18ACgkQ64mZXMKQ
wqnsaRAAr+Oqm2SvtNplR1oeYbJaTzoSFpfl3Nuog/bLfGT4myplA4M2lCH56nJL
/cVoFE/7Uj2y6KNfyRE2MUZoxvWCfZcOGa08M1DYCRWpiq4NBY/17plbSwKoXMSE
GATIY3IK+Yw0VmR7KzV88HFVz4OViolKoTetpe6d+oDgWZ3HWLHXS/RTGWd3QrJS
e+juaKxuN9goPSPS0W+ylSgKJJ3Vi+tsREGpCTHA5dbYxxa9WIybs+J9amSoi+G3
jQBKbzbd47RawgF3R8Eb2SnCRh3TN3OHnMhmfYChY7FmOadZHOMToF9J2ycrCqPO
jFUMJQoNru6IzuAT3fTT3/TVq9U7tcV+a6hNRcQuwVvczl3XfZaIC7loVamuKeoi
Dr5nuRIClqPjEDzSyd5Pa4rgwnSXlPxh/uOGs8biIjLn3PNiDU90N6MHYD+xvnuU
94Rtdv8lIfrFBKO1rjMic2f+RzDfUcbm6Tbl0KBeuqSMRzZxpkRzM2BtSXLTVgNp
Ye9KqQRi3SwDCtX6KsPC7wXpDZhdDu4KFjDWbQleQ2K9WmCoSdnaTRHqdyB27aHm
hzRaFY0ewyr+k5GiFBnTwpxRk9FF+W+AbF0Ei5ICjSGh2N/zEAXwsZaQeB15WXnB
2C02Ho74UMnyis17Lq/7HU+GE8tNZvNE7NeykZOkfk232DMsKaA=
=xfLH
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.