Repository navigation
Bug in w3c/epubcheck/src/main/java/com/adobe/epubcheck/util/PathUtil.java #1181
Description
It is valid to have user.dir value set to root ("/") (and this happens when subprocesses are used to launch the epubcheck via the commandline) but this routine then breaks all paths:
static final String workingDirectory = System.getProperty("user.dir");
public static String removeWorkingDirectory(String path)
{
if (path == null || path.length() == 0)
{
return path;
}
return path.replace(workingDirectory, ".");
}
By replacing all "/" chars in path to "." chars. Given "." chars are valid in filenames and directory names, this replacement is unfixable.
Please make this routine robust to all possible legal values for user.dir
Activity
rdeltour commentedon Aug 24, 2020
Thanks for the report 👍, I'll look into it tentatively for the next maintenance release!
fix: remove the user directory only at the start of paths (in messages)
fix: remove the user directory only at the start of paths (in messages)