Bug 50118 - java.lang.IllegalArgumentException: components cannot contain null or empty strings
Summary: java.lang.IllegalArgumentException: components cannot contain null or empty s...
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: POIFS (show other bugs)
Version: 3.5-FINAL
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-19 18:55 UTC by Raj Nagappan
Modified: 2010-10-24 18:12 UTC (History)
0 users



Attachments
an OLE binary file that creates the exception upon load (8.00 KB, application/octet-stream)
2010-10-19 18:55 UTC, Raj Nagappan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Raj Nagappan 2010-10-19 18:55:56 UTC
Created attachment 26184 [details]
an OLE binary file that creates the exception upon load

When I try to open a OLE binary with POI I get the following error message. The binaries are generated outside us so we have no control over them - we have to accept what we receive.

java.lang.IllegalArgumentException: components cannot contain null or empty strings
	at org.apache.poi.poifs.filesystem.POIFSDocumentPath.<init>(POIFSDocumentPath.java:131)
	at org.apache.poi.poifs.filesystem.DirectoryNode.<init>(DirectoryNode.java:72)
	at org.apache.poi.poifs.filesystem.DirectoryNode.createDirectory(DirectoryNode.java:330)
	at org.apache.poi.poifs.filesystem.POIFSFileSystem.processProperties(POIFSFileSystem.java:516)
	at org.apache.poi.poifs.filesystem.POIFSFileSystem.<init>(POIFSFileSystem.java:176)

A sample file that creates the error is attached.
Comment 1 Nick Burch 2010-10-19 18:57:56 UTC
Is the problem still there with poi 3.7 beta 3? There has been some poifs work since 3.5

Also, what is generating the problem file? Office or something else?
Comment 2 Raj Nagappan 2010-10-19 22:42:35 UTC
Yes it still occurs in 3.7 beta 3. It was created by embedded an object into a Lotus Notes memo file.
Comment 3 Nick Burch 2010-10-22 13:24:44 UTC
Fixed in r1026412 - while I was sure it wasn't allowed to have an empty path, it turns out it is, so we now support that.
Comment 4 Raj Nagappan 2010-10-24 18:12:06 UTC
Thanks for this :-)