udf: improve error management in udf_CS0toUTF8()
udf_CS0toUTF8() now returns -EINVAL on error. udf_load_pvoldesc() and udf_get_filename() do the same. Suggested-by: Jan Kara <jack@suse.cz> Signed-off-by: Fabian Frederick <fabf@skynet.be> Signed-off-by: Jan Kara <jack@suse.cz>
This commit is contained in:

committed by
Jan Kara

parent
d67e4a4814
commit
e9d4cf411f
@@ -927,17 +927,22 @@ static int udf_load_pvoldesc(struct super_block *sb, sector_t block)
|
||||
#endif
|
||||
}
|
||||
|
||||
if (!udf_build_ustr(instr, pvoldesc->volIdent, 32))
|
||||
if (udf_CS0toUTF8(outstr, instr)) {
|
||||
strncpy(UDF_SB(sb)->s_volume_ident, outstr->u_name,
|
||||
outstr->u_len > 31 ? 31 : outstr->u_len);
|
||||
udf_debug("volIdent[] = '%s'\n",
|
||||
UDF_SB(sb)->s_volume_ident);
|
||||
}
|
||||
if (!udf_build_ustr(instr, pvoldesc->volIdent, 32)) {
|
||||
ret = udf_CS0toUTF8(outstr, instr);
|
||||
if (ret < 0)
|
||||
goto out_bh;
|
||||
|
||||
strncpy(UDF_SB(sb)->s_volume_ident, outstr->u_name,
|
||||
outstr->u_len > 31 ? 31 : outstr->u_len);
|
||||
udf_debug("volIdent[] = '%s'\n", UDF_SB(sb)->s_volume_ident);
|
||||
}
|
||||
|
||||
if (!udf_build_ustr(instr, pvoldesc->volSetIdent, 128))
|
||||
if (udf_CS0toUTF8(outstr, instr))
|
||||
udf_debug("volSetIdent[] = '%s'\n", outstr->u_name);
|
||||
ret = udf_CS0toUTF8(outstr, instr);
|
||||
if (ret < 0)
|
||||
goto out_bh;
|
||||
|
||||
udf_debug("volSetIdent[] = '%s'\n", outstr->u_name);
|
||||
|
||||
ret = 0;
|
||||
out_bh:
|
||||
|
Reference in New Issue
Block a user