Community
Participate
Working Groups
I'm getting an out of memory exception in the eclipse patch file parser trying to parse a large binary file (not actually a patch): java.lang.OutOfMemoryError at java.lang.StringBuffer.toString(StringBuffer.java:790) at org.eclipse.compare.internal.core.patch.LineReader.readLine(LineReader.java:165) at org.eclipse.compare.internal.core.patch.PatchReader.parse(PatchReader.java:185) at org.eclipse.compare.internal.core.patch.PatchReader.parse(PatchReader.java:105) at org.eclipse.compare.patch.ApplyPatchOperation.internalParsePatch(ApplyPatchOperation.java:215) at org.eclipse.compare.patch.ApplyPatchOperation.parsePatch(ApplyPatchOperation.java:99) Instead, the parser should fail gracefully when the buffer exceeds a certain size.
(In reply to comment #0) > the parser should fail gracefully when the buffer exceeds a certain size. Do you have any specific size in mind? The parser can handle larger files if you adjust JVM memory settings, but it won't help much if you're trying to parse a binary file. I think a better approach would be to abort as soon as we realize we're dealing with a binary. However, you will still get in trouble if you try to parse a proper patch as large as the binary.
Perhaps if there is an extremely large line it could be a clue that it is a binary file. Maybe it is unrealistic for a line to exceed something like 1MB.
(In reply to comment #2) > Perhaps if there is an extremely large line it could be a clue that it is a > binary file. Maybe it is unrealistic for a line to exceed something like 1MB. Yup, we do a similar thing when guessing type in org.eclipse.compare.internal.CompareUIPlugin.guessType(ITypedElement). If any of the first 10 lines is longer than 1000 bytes we assume the input is not a text file.
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug. If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. -- The automated Eclipse Genie.
