Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 17 May 2021 09:55:50 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Kirk McKusick <mckusick@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 9a2fac6ba65f - main - Fix handling of embedded symbolic links (and history lesson).
Message-ID:  <YKITdkjjdbNc3Uhc@kib.kiev.ua>
In-Reply-To: <202105170000.14H00NgY054353@gitrepo.freebsd.org>
References:  <202105170000.14H00NgY054353@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, May 17, 2021 at 12:00:23AM +0000, Kirk McKusick wrote:
> diff --git a/sys/sys/mount.h b/sys/sys/mount.h
> index a6d750a1ff37..f341370ecd86 100644
> --- a/sys/sys/mount.h
> +++ b/sys/sys/mount.h
> @@ -224,7 +224,7 @@ struct mount {
>  	int		mnt_writeopcount;	/* (i) write syscalls pending */
>  	struct vfsoptlist *mnt_opt;		/* current mount options */
>  	struct vfsoptlist *mnt_optnew;		/* new options passed to fs */
> -	int		mnt_maxsymlinklen;	/* max size of short symlink */
> +	uint64_t	mnt_maxsymlinklen;	/* max size of short symlink */
Thank you for the interesting reading in the commit message.

Somewhat (un)related question is, why maxsymlinklen is put into struct mount?
Shouldn't it be contained to struct ufsmount and similar ext2 data structure.
The value is private to UFS, it has no use outside sys/ufs.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YKITdkjjdbNc3Uhc>