Thank you Sherman. -Jaikiran
On 05/10/18 9:59 PM, Xueming Shen wrote: > Hi Jaikiran, > > "wrap to throw a IOE" appears good. I will sponsor this patch. > > Thanks! > -Sherman > > On 10/5/18, 8:31 AM, Jaikiran Pai wrote: >> Thank you Chris. >> >> I've attached a patch here which catches the InvalidPathException within >> the implementation of ZipFile and wraps and throws a IOException. It's >> the same patch which I mentioned earlier in this thread, but now also >> includes jtreg testcase to reproduce the issue and verify the fix. I >> have run the jtreg test with and without this patch on my *nix system >> and it behaves as expected (fails without the fix and passes with it). I >> don't have a Windows OS at hand to test this patch and the testcase >> on it. >> >> I also did a very basic check and went through the JarFile/ZipFile >> constructor code path to see if there are other places in that flow >> which might need similar treatment. My not-so-thorough check didn't show >> up any other such instance. >> >> Could I please get help from someone who can sponsor this patch and help >> with the review too? >> >> -Jaikiran >> >> >> On 05/10/18 6:21 PM, Chris Hegarty wrote: >>>> On 5 Oct 2018, at 12:08, Jaikiran Pai<jai.forums2...@gmail.com> >>>> wrote: >>>> >>>>> I don't have access to create an issue for this in OpenJDK JIRA, >>>>> so I'll >>>>> go ahead and create one at bugs.java.com. >>>>> >>>> I just submitted the report. Internal review id 9057522 has been >>>> assigned to the issue. >>> And now in the JDK project: >>> >>> https://bugs.openjdk.java.net/browse/JDK-8211765 >>> >>> -Chris. >