Skip to content

Handle Malformed Classes Exception #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
devmehtabd merged 2 commits into from
Dec 1, 2025
Merged

Handle Malformed Classes Exception #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1255042
Handle Malformed Classes Exception
devmehtabd Nov 21, 2025
8f2cb64
Address Out of Memory exeption
devmehtabd Nov 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions method-analyzer-core/src/main/java/com/blackduck/method/analyzer/core/MethodUseAnalyzer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,9 @@ public Path analyze(Path sourceDirectory, Path outputDirectory, String outputFil
} else {
throw e;
}
} catch (IndexOutOfBoundsException | ClassFormatError | NegativeArraySizeException e) {
//IDETECT-4924 Handle malformed classes properly
brokenFiles.put(classFile, "Malformed class structure: " + Strings.nullToEmpty(e.getMessage()));
}
}

Expand Down
4 changes: 2 additions & 2 deletions ...rc/main/java/com/blackduck/method/analyzer/core/bytecode/ClassMethodReferenceVisitor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,12 +221,12 @@ private Type getReturnType(String descriptor) {

try {
return Type.getType(descriptor);
} catch (StringIndexOutOfBoundsException e2) {
} catch (StringIndexOutOfBoundsException | IllegalArgumentException e2) {
// IDETECT-3909 This can occur for malformed signatures which are just the type, not the method, for
// an unknown reason
logger.warn("Malformed method descriptor, skipping reference processing: {}:{} ({})", currentClassName, currentMethodName, descriptor);

return Type.getType(descriptor);
return Type.getType(Object.class);
}
}
}
Expand Down