The following document contains the results of FindBugs
FindBugs Version is 3.0.0
Threshold is medium
Effort is min
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.api.ApplyCommand.apply(File, FileHeader): new java.io.FileWriter(File) | I18N | DM_DEFAULT_ENCODING | 247 | High |
Found reliance on default encoding in org.eclipse.jgit.api.ApplyCommand.apply(File, FileHeader): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 202 | High |
org.eclipse.jgit.api.ApplyCommand.apply(File, FileHeader) may fail to clean up java.io.Writer on checked exception | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE | 247 | Medium |
Switch statement found in org.eclipse.jgit.api.ApplyCommand.apply(File, FileHeader) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 209-229 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.api.GarbageCollectCommand.setExpire(Date) may expose internal representation by storing an externally mutable object into GarbageCollectCommand.expire | MALICIOUS_CODE | EI_EXPOSE_REP2 | 129 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.api.MergeResult.getMergedCommits() may expose internal representation by returning MergeResult.mergedCommits | MALICIOUS_CODE | EI_EXPOSE_REP | 372 | Medium |
new org.eclipse.jgit.api.MergeResult(ObjectId, ObjectId, ObjectId[], MergeResult$MergeStatus, MergeStrategy, Map, Map, String) may expose internal representation by storing an externally mutable object into MergeResult.mergedCommits | MALICIOUS_CODE | EI_EXPOSE_REP2 | 330 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.createNewFile() ignored in org.eclipse.jgit.api.RebaseCommand.calculatePickList(RevCommit) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 1174 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.api.RebaseCommand.processStep(RebaseTodoLine, boolean) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 484 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.api.RebaseCommand.squashIntoPrevious(boolean, RebaseTodoLine) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 780 | Medium |
Switch statement found in org.eclipse.jgit.api.RebaseCommand.processStep(RebaseTodoLine, boolean) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 474-477 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.eclipse.jgit.api.RevertCommand.call() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 203 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.diff.RawText(byte[]) may expose internal representation by storing an externally mutable object into RawText.content | MALICIOUS_CODE | EI_EXPOSE_REP2 | 91 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.eclipse.jgit.dircache.DirCacheCheckout.processEntry(CanonicalTreeParser, CanonicalTreeParser, DirCacheBuildIterator, WorkingTreeIterator) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 810 | Medium |
Load of known null value in org.eclipse.jgit.dircache.DirCacheCheckout.processEntry(CanonicalTreeParser, CanonicalTreeParser, DirCacheBuildIterator, WorkingTreeIterator) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 878 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.dircache.DirCacheEntry(byte[], int) may expose internal representation by storing an externally mutable object into DirCacheEntry.path | MALICIOUS_CODE | EI_EXPOSE_REP2 | 286 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.dircache.DirCacheIterator.<static initializer for DirCacheIterator>(): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 76 | High |
org.eclipse.jgit.dircache.DirCacheIterator.idBuffer() may expose internal representation by returning DirCacheIterator.zeroid | MALICIOUS_CODE | EI_EXPOSE_REP | 167 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.gitrepo.ManifestParser.startElement(String, String, String, Attributes) may fail to clean up java.io.InputStream | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION | 228 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.hooks.GitHook.doRun(): java.io.ByteArrayOutputStream.toString() | I18N | DM_DEFAULT_ENCODING | 154 | High |
Found reliance on default encoding in org.eclipse.jgit.hooks.GitHook.doRun(): new java.io.PrintStream(OutputStream) | I18N | DM_DEFAULT_ENCODING | 149 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Synchronization performed on java.util.concurrent.ConcurrentHashMap in org.eclipse.jgit.internal.storage.dfs.DfsBlockCache.getOrCreate(DfsPackDescription, DfsPackKey) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 271 | Medium |
Synchronization performed on java.util.concurrent.ConcurrentHashMap in org.eclipse.jgit.internal.storage.dfs.DfsBlockCache.remove(DfsPackFile) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 511 | Medium |
org.eclipse.jgit.internal.storage.dfs.DfsBlockCache.creditSpace(int) does not release lock on all exception paths | MT_CORRECTNESS | UL_UNRELEASED_LOCK_EXCEPTION_PATH | 422 | Medium |
Increment of volatile field org.eclipse.jgit.internal.storage.dfs.DfsBlockCache.statEvict in org.eclipse.jgit.internal.storage.dfs.DfsBlockCache.reserveSpace(int) | MT_CORRECTNESS | VO_VOLATILE_INCREMENT | 411 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.StoredObjectRepresentation to org.eclipse.jgit.internal.storage.dfs.DfsObjectRepresentation in org.eclipse.jgit.internal.storage.dfs.DfsCachedPack.hasObject(ObjectToPack, StoredObjectRepresentation) | STYLE | BC_UNCONFIRMED_CAST | 78 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.storage.dfs.DfsObjDatabase.scanPacks(DfsObjDatabase$PackList) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 339 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.StoredObjectRepresentation to org.eclipse.jgit.internal.storage.dfs.DfsObjectRepresentation in org.eclipse.jgit.internal.storage.dfs.DfsObjectToPack.select(StoredObjectRepresentation) | STYLE | BC_UNCONFIRMED_CAST | 83 | Medium |
Class org.eclipse.jgit.internal.storage.dfs.DfsObjectToPack defines non-transient non-serializable instance field pack | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to len in org.eclipse.jgit.internal.storage.dfs.DfsPackFile.readOneBlock(long, DfsReader) | STYLE | DLS_DEAD_LOCAL_STORE | 816 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.ObjectToPack to org.eclipse.jgit.internal.storage.dfs.DfsObjectToPack in org.eclipse.jgit.internal.storage.dfs.DfsReader.copyObjectAsIs(PackOutputStream, ObjectToPack, boolean) | STYLE | BC_UNCONFIRMED_CAST | 490 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.eclipse.jgit.internal.storage.dfs.InMemoryRepository$MemObjDatabase be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 80-132 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.dfs.InMemoryRepository$MemPack doesn't override DfsPackDescription.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 1 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Synchronization performed on java.util.concurrent.ConcurrentHashMap in org.eclipse.jgit.internal.storage.dfs.InMemoryRepository$MemRefDatabase.compareAndPut(Ref, Ref) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 272 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.file.GC.setExpire(Date) may expose internal representation by storing an externally mutable object into GC.expire | MALICIOUS_CODE | EI_EXPOSE_REP2 | 989 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.internal.storage.file.GC.prune(Set) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 423 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.internal.storage.file.GC.writePack(Set, Set, Set, List) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 825 | Medium |
Exceptional return value of java.io.File.setReadOnly() ignored in org.eclipse.jgit.internal.storage.file.GC.writePack(Set, Set, Set, List) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 788 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.eclipse.jgit.internal.storage.file.GC$RepoStatistics be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 842-889 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.StoredObjectRepresentation to org.eclipse.jgit.internal.storage.file.LocalObjectRepresentation in org.eclipse.jgit.internal.storage.file.LocalCachedPack.hasObject(ObjectToPack, StoredObjectRepresentation) | STYLE | BC_UNCONFIRMED_CAST | 91 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.StoredObjectRepresentation to org.eclipse.jgit.internal.storage.file.LocalObjectRepresentation in org.eclipse.jgit.internal.storage.file.LocalObjectToPack.select(StoredObjectRepresentation) | STYLE | BC_UNCONFIRMED_CAST | 73 | Medium |
Class org.eclipse.jgit.internal.storage.file.LocalObjectToPack defines non-transient non-serializable instance field pack | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.setLastModified(long) ignored in org.eclipse.jgit.internal.storage.file.LockFile.waitForStatChange() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 417 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.internal.storage.file.ObjectDirectory.open(File): new java.io.FileReader(File) | I18N | DM_DEFAULT_ENCODING | 892 | High |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.storage.file.ObjectDirectory.myAlternates() | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 861 | Medium |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.storage.file.ObjectDirectory.scanPacks(ObjectDirectory$PackList) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 741 | Medium |
Exceptional return value of java.io.File.setReadOnly() ignored in org.eclipse.jgit.internal.storage.file.ObjectDirectory.insertUnpackedObject(File, ObjectId, boolean) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 611 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.setReadOnly() ignored in org.eclipse.jgit.internal.storage.file.ObjectDirectoryPackParser.parse(ProgressMonitor, ProgressMonitor) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 203 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.eclipse.jgit.internal.storage.file.PackBitmapIndexRemapper$Entry be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 199-209 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.file.PackFile.mmap(long, int) forces garbage collection; extremely dubious except in benchmarking code | PERFORMANCE | DM_GC | 685 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.file.PackIndexWriter.write(List, byte[]) may expose internal representation by storing an externally mutable object into PackIndexWriter.packChecksum | MALICIOUS_CODE | EI_EXPOSE_REP2 | 211 | Medium |
org.eclipse.jgit.internal.storage.file.PackIndexWriter.TOC should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 70 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unchecked/unconfirmed cast from org.eclipse.jgit.internal.storage.pack.ObjectToPack to org.eclipse.jgit.internal.storage.file.LocalObjectToPack in org.eclipse.jgit.internal.storage.file.WindowCursor.copyObjectAsIs(PackOutputStream, ObjectToPack, boolean) | STYLE | BC_UNCONFIRMED_CAST | 188 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.internal.storage.pack.DeltaIndex(byte[]) may expose internal representation by storing an externally mutable object into DeltaIndex.src | MALICIOUS_CODE | EI_EXPOSE_REP2 | 133 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.pack.DeltaTask$WeightedPath defines compareTo(DeltaTask$WeightedPath) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 231-234 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.eclipse.jgit.internal.storage.pack.ObjectToPack defines non-transient non-serializable instance field cachedDelta | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Public static org.eclipse.jgit.internal.storage.pack.PackExt.values() may expose internal representation by returning PackExt.VALUES | MALICIOUS_CODE | MS_EXPOSE_REP | 61 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.pack.PackOutputStream.getCopyBuffer() may expose internal representation by returning PackOutputStream.copyBuffer | MALICIOUS_CODE | EI_EXPOSE_REP | 229 | Medium |
Potentially dangerous use of non-short-circuit logic in org.eclipse.jgit.internal.storage.pack.PackOutputStream.writeHeader(ObjectToPack, long) | STYLE | NS_DANGEROUS_NON_SHORT_CIRCUIT | 186 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread field: org.eclipse.jgit.internal.storage.pack.PackWriterBitmapPreparer.maxCommits; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 97 | Medium |
Unread field: org.eclipse.jgit.internal.storage.pack.PackWriterBitmapPreparer.minCommits; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 96 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.pack.PackWriterBitmapPreparer$WalkResult$1.next() can't throw NoSuchElementException | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 365 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.CommitBuilder.getParentIds() may expose internal representation by returning CommitBuilder.parentIds | MALICIOUS_CODE | EI_EXPOSE_REP | 144 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.lib.Config.getLong(String, String, String, long) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 263-271 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.ConfigSnapshot$LineComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 219-221 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Public static org.eclipse.jgit.lib.Constants.encodedTypeString(int) may expose internal representation by returning Constants.ENCODED_TYPE_BLOB | MALICIOUS_CODE | MS_EXPOSE_REP | 452 | Medium |
Public static org.eclipse.jgit.lib.Constants.encodedTypeString(int) may expose internal representation by returning Constants.ENCODED_TYPE_COMMIT | MALICIOUS_CODE | MS_EXPOSE_REP | 448 | Medium |
Public static org.eclipse.jgit.lib.Constants.encodedTypeString(int) may expose internal representation by returning Constants.ENCODED_TYPE_TAG | MALICIOUS_CODE | MS_EXPOSE_REP | 454 | Medium |
Public static org.eclipse.jgit.lib.Constants.encodedTypeString(int) may expose internal representation by returning Constants.ENCODED_TYPE_TREE | MALICIOUS_CODE | MS_EXPOSE_REP | 450 | Medium |
org.eclipse.jgit.lib.Constants.PACK_SIGNATURE is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 224 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.ObjectChecker.author is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 82 | Medium |
org.eclipse.jgit.lib.ObjectChecker.committer is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 85 | Medium |
org.eclipse.jgit.lib.ObjectChecker.encoding is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 88 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tagger is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 100 | Medium |
org.eclipse.jgit.lib.ObjectChecker.object should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 91 | Medium |
org.eclipse.jgit.lib.ObjectChecker.parent should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 79 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tag should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 97 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tree should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 76 | Medium |
org.eclipse.jgit.lib.ObjectChecker.type should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 94 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.ObjectLoader$SmallObject.getCachedBytes() may expose internal representation by returning ObjectLoader$SmallObject.data | MALICIOUS_CODE | EI_EXPOSE_REP | 318 | Medium |
new org.eclipse.jgit.lib.ObjectLoader$SmallObject(int, byte[]) may expose internal representation by storing an externally mutable object into ObjectLoader$SmallObject.data | MALICIOUS_CODE | EI_EXPOSE_REP2 | 298 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.lib.ObjectStream$SmallStream(int, byte[]) may expose internal representation by storing an externally mutable object into ObjectStream$SmallStream.data | MALICIOUS_CODE | EI_EXPOSE_REP2 | 93 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.lib.RebaseTodoFile.parseLine(byte[], int, int): new String(byte[], int, int) | I18N | DM_DEFAULT_ENCODING | 180 | High |
Redundant nullcheck of action, which is known to be non-null in org.eclipse.jgit.lib.RebaseTodoFile.parseLine(byte[], int, int) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 184 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.RefDatabase.SEARCH_PATH should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 70 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in new org.eclipse.jgit.lib.TextProgressMonitor(): new java.io.PrintWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 59 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.lib.TreeEntry.getFullNameUTF8(): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 214 | High |
org.eclipse.jgit.lib.TreeEntry.getNameUTF8() may expose internal representation by returning TreeEntry.nameUTF8 | MALICIOUS_CODE | EI_EXPOSE_REP | 119 | Medium |
org.eclipse.jgit.lib.TreeEntry.rename(byte[]) may expose internal representation by storing an externally mutable object into TreeEntry.nameUTF8 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 152 | Medium |
Possible null pointer dereference of TreeEntry.id in org.eclipse.jgit.lib.TreeEntry.setId(ObjectId) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 191 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.merge.ResolveMerger.getCommitNames() may expose internal representation by returning ResolveMerger.commitNames | MALICIOUS_CODE | EI_EXPOSE_REP | 885 | Medium |
org.eclipse.jgit.merge.ResolveMerger.setCommitNames(String[]) may expose internal representation by storing an externally mutable object into ResolveMerger.commitNames | MALICIOUS_CODE | EI_EXPOSE_REP2 | 877 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.eclipse.jgit.merge.ResolveMerger.writeMergedFile(MergeResult) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 798 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.patch.FileHeader.getBuffer() may expose internal representation by returning FileHeader.buf | MALICIOUS_CODE | EI_EXPOSE_REP | 169 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.patch.FormatError.getBuffer() may expose internal representation by returning FormatError.buf | MALICIOUS_CODE | EI_EXPOSE_REP | 88 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.revwalk.ObjectWalk.getPathBuffer() may expose internal representation by returning ObjectWalk.pathBuf | MALICIOUS_CODE | EI_EXPOSE_REP | 624 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.revwalk.RevCommit.getParents() may expose internal representation by returning RevCommit.parents | MALICIOUS_CODE | EI_EXPOSE_REP | 362 | Medium |
org.eclipse.jgit.revwalk.RevCommit.getRawBuffer() may expose internal representation by returning RevCommit.buffer | MALICIOUS_CODE | EI_EXPOSE_REP | 378 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.transport.AmazonS3.error(String, String, HttpURLConnection): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 542 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.transport.CredentialItem$CharArrayType.getValue() may expose internal representation by returning CredentialItem$CharArrayType.value | MALICIOUS_CODE | EI_EXPOSE_REP | 184 | Medium |
org.eclipse.jgit.transport.CredentialItem$CharArrayType.setValueNoCopy(char[]) may expose internal representation by storing an externally mutable object into CredentialItem$CharArrayType.value | MALICIOUS_CODE | EI_EXPOSE_REP2 | 210 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.transport.FetchProcess.updateFETCH_HEAD(FetchResult): new java.io.OutputStreamWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 321 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.transport.HttpTransport.connectionFactory should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 62 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.transport.JschConfigSessionFactory.getSession(URIish, CredentialsProvider, FS, int) calls Thread.sleep() with a lock held | MT_CORRECTNESS | SWL_SLEEP_WITH_LOCK_HELD | 139 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.transport.NetRC.parse(): new java.io.FileReader(File) | I18N | DM_DEFAULT_ENCODING | 208 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.transport.OpenSshConfig.parse(InputStream): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 179 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.eclipse.jgit.transport.PackParser$DeltaChain defines non-transient non-serializable instance field head | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of org.eclipse.jgit.transport.PushCertificateParser.pusher, which is known to be non-null in org.eclipse.jgit.transport.PushCertificateParser.receiveHeader(PushCertificateParser$StringReader, boolean) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 376 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.transport.SideBandInputStream.doProgressLine(String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 240 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.transport.UsernamePasswordCredentialsProvider(String, char[]) may expose internal representation by storing an externally mutable object into UsernamePasswordCredentialsProvider.password | MALICIOUS_CODE | EI_EXPOSE_REP2 | 76 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.transport.WalkFetchConnection.close() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 259 | Medium |
Switch statement found in org.eclipse.jgit.transport.WalkFetchConnection.markLocalObjComplete(RevObject) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 704-712 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.treewalk.AbstractTreeIterator.getEntryPathBuffer() may expose internal representation by returning AbstractTreeIterator.path | MALICIOUS_CODE | EI_EXPOSE_REP | 437 | Medium |
org.eclipse.jgit.treewalk.AbstractTreeIterator.zeroid should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 87 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.treewalk.CanonicalTreeParser.idBuffer() may expose internal representation by returning CanonicalTreeParser.raw | MALICIOUS_CODE | EI_EXPOSE_REP | 252 | Medium |
org.eclipse.jgit.treewalk.CanonicalTreeParser.reset(byte[]) may expose internal representation by storing an externally mutable object into CanonicalTreeParser.raw | MALICIOUS_CODE | EI_EXPOSE_REP2 | 127 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.treewalk.EmptyTreeIterator.idBuffer() may expose internal representation by returning EmptyTreeIterator.zeroid | MALICIOUS_CODE | EI_EXPOSE_REP | 107 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.treewalk.WorkingTreeIterator.idBuffer() may expose internal representation by returning WorkingTreeIterator.contentId | MALICIOUS_CODE | EI_EXPOSE_REP | 260 | Medium |
org.eclipse.jgit.treewalk.WorkingTreeIterator.idBuffer() may expose internal representation by returning WorkingTreeIterator.zeroid | MALICIOUS_CODE | EI_EXPOSE_REP | 290 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.util.Base64.decode(String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 311 | High |
Found reliance on default encoding in org.eclipse.jgit.util.Base64.encodeBytes(byte[], int, int): new String(byte[], int, int) | I18N | DM_DEFAULT_ENCODING | 190 | High |
Switch statement found in org.eclipse.jgit.util.Base64.encode3to4(byte[], int, int, byte[], int) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 114-118 | Medium |
Switch statement found in org.eclipse.jgit.util.Base64.encode3to4(byte[], int, int, byte[], int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 112-122 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.eclipse.jgit.util.FS.runProcess(ProcessBuilder, OutputStream, OutputStream, String): new java.io.PrintWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 912 | High |
Incorrect lazy initialization of static field org.eclipse.jgit.util.FS.factory in org.eclipse.jgit.util.FS.detect(Boolean) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 151-152 | Medium |
Exceptional return value of java.util.concurrent.ExecutorService.submit(Callable) ignored in org.eclipse.jgit.util.FS.runProcess(ProcessBuilder, OutputStream, OutputStream, String) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 909 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in new org.eclipse.jgit.util.FS$StreamGobbler(InputStream, OutputStream): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 1202 | High |
Found reliance on default encoding in new org.eclipse.jgit.util.FS$StreamGobbler(InputStream, OutputStream): new java.io.OutputStreamWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 1206 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dereference of the result of readLine() without nullcheck in org.eclipse.jgit.util.FS_POSIX.readUmask() | STYLE | NP_DEREFERENCE_OF_READLINE_VALUE | 125 | Medium |
Exception is caught when Exception is not thrown in org.eclipse.jgit.util.FS_POSIX.readUmask() | STYLE | REC_CATCH_EXCEPTION | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.util.FS_Win32.detectSymlinkSupport() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 173 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Public static org.eclipse.jgit.util.GitDateParser.parse(String, Calendar, Locale) may expose internal representation by returning GitDateParser.NEVER | MALICIOUS_CODE | MS_EXPOSE_REP | 219 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.util.IO.readWholeStream(InputStream, int) may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 221 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.util.QuotedString$BourneStyle.quote(String) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 143-151 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.util.RawCharSequence(byte[], int, int) may expose internal representation by storing an externally mutable object into RawCharSequence.buffer | MALICIOUS_CODE | EI_EXPOSE_REP2 | 72 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.util.RawParseUtils.parseBase10(byte[], int, MutableInteger) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 214-220 | Medium |
Switch statement found in org.eclipse.jgit.util.RawParseUtils.parseLongBase10(byte[], int, MutableInteger) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 268-274 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Write to static field org.eclipse.jgit.util.SystemReader.isMacOS from instance method org.eclipse.jgit.util.SystemReader.isMacOS() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 316 | Medium |
Write to static field org.eclipse.jgit.util.SystemReader.isWindows from instance method org.eclipse.jgit.util.SystemReader.isWindows() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 305 | Medium |