ceph: validate snapdirname option length when mounting
commit 12eb22a5a609421b380c3c6ca887474fb2089b2c upstream. It becomes a path component, so it shouldn't exceed NAME_MAX characters. This was hardened in commit c152737be22b ("ceph: Use strscpy() instead of strcpy() in __get_snap_name()"), but no actual check was put in place. Cc: stable@vger.kernel.org Signed-off-by: Ilya Dryomov <idryomov@gmail.com> Reviewed-by: Alex Markuze <amarkuze@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
6e865e3b42
commit
f698a9998c
1 changed files with 2 additions and 0 deletions
|
@ -301,6 +301,8 @@ static int ceph_parse_mount_param(struct fs_context *fc,
|
||||||
|
|
||||||
switch (token) {
|
switch (token) {
|
||||||
case Opt_snapdirname:
|
case Opt_snapdirname:
|
||||||
|
if (strlen(param->string) > NAME_MAX)
|
||||||
|
return invalfc(fc, "snapdirname too long");
|
||||||
kfree(fsopt->snapdir_name);
|
kfree(fsopt->snapdir_name);
|
||||||
fsopt->snapdir_name = param->string;
|
fsopt->snapdir_name = param->string;
|
||||||
param->string = NULL;
|
param->string = NULL;
|
||||||
|
|
Loading…
Add table
Reference in a new issue