On Apr 8, 2014, at 8:43 PM, Hin-Tak Leung wrote: [snip] >> Yes, it is possible to split the work on several patches. I agree with >> it. >> >> But I suppose that these patches needs to be joined in one patchset. >> Because reported issue can be solved by fix as for file names as for >> xattr names. >> >> Of course, it is possible to fix the issue with file names, firstly, and >> then the issue with xattr names. But it doesn't make sense to mix fixes >> for file names and for xattr names in one patch. Because partial fix in >> hfsplus_listxattr() doesn't fix the issue for xattr names. >> > > Think of it as "fixes worst-case usage of hfsplus_uni2asc()", > then it makes sense to have this as one patch. Allowing full usage > of getting and setting of long multi-lingual attributes, and checking > other uses of HFSPLUS_ATTR_MAX_STRLEN really need to be another > (multiple others). > > There are only two usages of hfsplus_uni2asc(), one for names and > one for attributes. Unfortunately, it doesn't make sense to have fix only in hfsplus_listxattr(). It fixes nothing for xattr names. Of course, you can mention name of the patch. :) But such partial fix doesn't make sense for me. It will be more clear way to have one patch for file names and another patch for xattr names. Thanks, Vyacheslav Dubeyko. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
