Commit 525e2c56 authored by Andrew Gabbasov's avatar Andrew Gabbasov Committed by Jan Kara

udf: Parameterize output length in udf_put_filename

Make the desired output length a parameter rather than have it
hard-coded to UDF_NAME_LEN. Although all call sites still have
this length the same, this parameterization will make the function
more universal and also consistent with udf_get_filename.
Signed-off-by: default avatarAndrew Gabbasov <andrew_gabbasov@mentor.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 7955118e
...@@ -362,8 +362,9 @@ static struct fileIdentDesc *udf_add_entry(struct inode *dir, ...@@ -362,8 +362,9 @@ static struct fileIdentDesc *udf_add_entry(struct inode *dir,
*err = -EINVAL; *err = -EINVAL;
goto out_err; goto out_err;
} }
namelen = udf_put_filename(sb, dentry->d_name.name, name, namelen = udf_put_filename(sb, dentry->d_name.name,
dentry->d_name.len); dentry->d_name.len,
name, UDF_NAME_LEN);
if (!namelen) { if (!namelen) {
*err = -ENAMETOOLONG; *err = -ENAMETOOLONG;
goto out_err; goto out_err;
...@@ -997,8 +998,9 @@ static int udf_symlink(struct inode *dir, struct dentry *dentry, ...@@ -997,8 +998,9 @@ static int udf_symlink(struct inode *dir, struct dentry *dentry,
} }
if (pc->componentType == 5) { if (pc->componentType == 5) {
namelen = udf_put_filename(sb, compstart, name, namelen = udf_put_filename(sb, compstart,
symname - compstart); symname - compstart,
name, UDF_NAME_LEN);
if (!namelen) if (!namelen)
goto out_no_entry; goto out_no_entry;
......
...@@ -216,8 +216,8 @@ udf_get_lb_pblock(struct super_block *sb, struct kernel_lb_addr *loc, ...@@ -216,8 +216,8 @@ udf_get_lb_pblock(struct super_block *sb, struct kernel_lb_addr *loc,
/* unicode.c */ /* unicode.c */
extern int udf_get_filename(struct super_block *, uint8_t *, int, uint8_t *, extern int udf_get_filename(struct super_block *, uint8_t *, int, uint8_t *,
int); int);
extern int udf_put_filename(struct super_block *, const uint8_t *, uint8_t *, extern int udf_put_filename(struct super_block *, const uint8_t *, int,
int); uint8_t *, int);
extern int udf_build_ustr(struct ustr *, dstring *, int); extern int udf_build_ustr(struct ustr *, dstring *, int);
extern int udf_CS0toUTF8(struct ustr *, const struct ustr *); extern int udf_CS0toUTF8(struct ustr *, const struct ustr *);
......
...@@ -395,22 +395,22 @@ int udf_get_filename(struct super_block *sb, uint8_t *sname, int slen, ...@@ -395,22 +395,22 @@ int udf_get_filename(struct super_block *sb, uint8_t *sname, int slen,
return ret; return ret;
} }
int udf_put_filename(struct super_block *sb, const uint8_t *sname, int udf_put_filename(struct super_block *sb, const uint8_t *sname, int slen,
uint8_t *dname, int flen) uint8_t *dname, int dlen)
{ {
struct ustr unifilename; struct ustr unifilename;
int namelen; int namelen;
if (!udf_char_to_ustr(&unifilename, sname, flen)) if (!udf_char_to_ustr(&unifilename, sname, slen))
return 0; return 0;
if (UDF_QUERY_FLAG(sb, UDF_FLAG_UTF8)) { if (UDF_QUERY_FLAG(sb, UDF_FLAG_UTF8)) {
namelen = udf_UTF8toCS0(dname, &unifilename, UDF_NAME_LEN); namelen = udf_UTF8toCS0(dname, &unifilename, dlen);
if (!namelen) if (!namelen)
return 0; return 0;
} else if (UDF_QUERY_FLAG(sb, UDF_FLAG_NLS_MAP)) { } else if (UDF_QUERY_FLAG(sb, UDF_FLAG_NLS_MAP)) {
namelen = udf_NLStoCS0(UDF_SB(sb)->s_nls_map, dname, namelen = udf_NLStoCS0(UDF_SB(sb)->s_nls_map, dname,
&unifilename, UDF_NAME_LEN); &unifilename, dlen);
if (!namelen) if (!namelen)
return 0; return 0;
} else } else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment