The following document contains the results of SpotBugs
SpotBugs Version is 3.1.12
Threshold is medium
Effort is default
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.api.ApplyCommand.apply(File, FileHeader) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 220-244 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Return value of org.eclipse.jgit.revwalk.RevCommit.getTree() ignored, but method has no side effect | STYLE | RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT | 254 | 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 | 133 | 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 | 386 | 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 | 338 | 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 | 1206 | 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 | 497 | 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 | 800 | 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 | 487-490 | 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 | 211 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.eclipse.jgit.api.SubmoduleStatusCommand.getStatus(SubmoduleWalk) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 144 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.api.errors.FilterFailedException.getOutput() may expose internal representation by returning FilterFailedException.stdout | MALICIOUS_CODE | EI_EXPOSE_REP | 133 | Medium |
new org.eclipse.jgit.api.errors.FilterFailedException(int, String, String, byte[], String) may expose internal representation by storing an externally mutable object into FilterFailedException.stdout | MALICIOUS_CODE | EI_EXPOSE_REP2 | 104 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of t, which is known to be non-null in org.eclipse.jgit.attributes.AttributesHandler.rootOf(AbstractTreeIterator) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 436 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.eclipse.jgit.attributes.FilterCommand.in | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 80 | Medium |
Unread public/protected field: org.eclipse.jgit.attributes.FilterCommand.out | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 81 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.diff.RawText.getRawContent() may expose internal representation by returning RawText.content | MALICIOUS_CODE | EI_EXPOSE_REP | 135 | Medium |
new org.eclipse.jgit.diff.RawText(byte[], IntList) may expose internal representation by storing an externally mutable object into RawText.content | MALICIOUS_CODE | EI_EXPOSE_REP2 | 112 | 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 | 1042 | Medium |
Load of known null value in org.eclipse.jgit.dircache.DirCacheCheckout.processEntry(CanonicalTreeParser, CanonicalTreeParser, DirCacheBuildIterator, WorkingTreeIterator) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 1110 | 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 | 292 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.dircache.DirCacheIterator.idBuffer() may expose internal representation by returning DirCacheIterator.zeroid | MALICIOUS_CODE | EI_EXPOSE_REP | 171 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.errors.CheckoutConflictException.getConflictingFiles() may expose internal representation by returning CheckoutConflictException.conflicting | MALICIOUS_CODE | EI_EXPOSE_REP | 91 | Medium |
new org.eclipse.jgit.errors.CheckoutConflictException(String[]) may expose internal representation by storing an externally mutable object into CheckoutConflictException.conflicting | MALICIOUS_CODE | EI_EXPOSE_REP2 | 80 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.gitrepo.RepoCommand$RemoteFile.getContents() may expose internal representation by returning RepoCommand$RemoteFile.contents | MALICIOUS_CODE | EI_EXPOSE_REP | 243 | Medium |
new org.eclipse.jgit.gitrepo.RepoCommand$RemoteFile(byte[], FileMode) may expose internal representation by storing an externally mutable object into RepoCommand$RemoteFile.contents | MALICIOUS_CODE | EI_EXPOSE_REP2 | 229 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.ignore.IgnoreNode.checkIgnored(String, boolean) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 180 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.internal.ketch.KetchLeader.onReplicaUpdate(KetchReplica) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 518-523 | Medium |
org.eclipse.jgit.internal.ketch.KetchLeader.notifySuccess(Round) does not release lock on all exception paths | MT_CORRECTNESS | UL_UNRELEASED_LOCK_EXCEPTION_PATH | 549 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Format string should use %n rather than \n in org.eclipse.jgit.internal.ketch.LeaderSnapshot.toString() | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 144 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.ketch.Proposal.addListener(Runnable) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 288 | Medium |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.ketch.Proposal.await() | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 344 | Medium |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.ketch.Proposal.await(long, TimeUnit) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 363 | Medium |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.ketch.Proposal.awaitStateChange(Proposal$State, long, TimeUnit) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 387 | Medium |
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.ketch.Proposal.notifyState(Proposal$State) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 397 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to len in org.eclipse.jgit.internal.storage.dfs.BlockBasedFile.readOneBlock(long, DfsReader, ReadableChannel) | STYLE | DLS_DEAD_LOCAL_STORE | 190 | 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.DfsCachedPack.hasObject(ObjectToPack, StoredObjectRepresentation) | STYLE | BC_UNCONFIRMED_CAST | 87 | 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 | 604 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.dfs.DfsObjDatabase$PackSource$PackSourceComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 1-232 | 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 | 85 | 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 |
---|---|---|---|---|
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 | 669 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.dfs.DfsStreamKey$ByteArrayDfsStreamKey defines equals but not hashCode | BAD_PRACTICE | HE_EQUALS_NO_HASHCODE | 123-128 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.dfs.DfsStreamKey$ForReverseIndex defines equals but not hashCode | BAD_PRACTICE | HE_EQUALS_NO_HASHCODE | 142 | 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 |
---|---|---|---|---|
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 | 1473 | Medium |
org.eclipse.jgit.internal.storage.file.GC.setPackExpire(Date) may expose internal representation by storing an externally mutable object into GC.packExpire | MALICIOUS_CODE | EI_EXPOSE_REP2 | 1487 | Medium |
Exceptional return value of java.util.concurrent.ExecutorService.submit(Callable) ignored in org.eclipse.jgit.internal.storage.file.GC.gc() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 285 | 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 | 673 | Medium |
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.internal.storage.file.GC.writePack(Set, Set, Set, Set, List) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 1281 | Medium |
Exceptional return value of java.io.File.setReadOnly() ignored in org.eclipse.jgit.internal.storage.file.GC.writePack(Set, Set, Set, Set, List) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 1235 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.delete() ignored in org.eclipse.jgit.internal.storage.file.GcLog.commit() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 156 | 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 | 93 | 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 | 75 | 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 |
---|---|---|---|---|
Synchronization performed on java.util.concurrent.atomic.AtomicReference in org.eclipse.jgit.internal.storage.file.ObjectDirectory.myAlternates() | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 1003 | 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 | 866 | 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 | 724 | 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 | 210 | 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 | 776 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.file.PackFileSnapshot doesn't override FileSnapshot.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 1 | Medium |
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 | 69 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exceptional return value of java.io.File.setReadOnly() ignored in org.eclipse.jgit.internal.storage.file.PackInserter.flush() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 282 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.internal.storage.file.PackedBatchRefUpdate.applyUpdates(RevWalk, RefList, List) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 378-383 | 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 | 220 | 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 | 134 | 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 | 245-249 | 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 | 73 | 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 | 241 | 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 | 194 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Result of integer multiplication cast to long in new org.eclipse.jgit.internal.storage.pack.PackWriterBitmapPreparer(ObjectReader, PackBitmapIndexBuilder, ProgressMonitor, Set, PackConfig) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.internal.storage.pack.PackWriterBitmapPreparer$CommitSelectionHelper$1.next() can't throw NoSuchElementException | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 574 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.eclipse.jgit.internal.storage.reftable.ReftableWriter$RefList defines non-transient non-serializable instance field blockPos | BAD_PRACTICE | SE_BAD_FIELD | Not available | 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 | 205 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Comparison of String parameter using == or != in org.eclipse.jgit.lib.Config.isMissing(String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 140 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.ConfigSnapshot$LineComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 1-224 | 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 | 524 | 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 | 520 | 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 | 526 | 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 | 522 | Medium |
org.eclipse.jgit.lib.Constants.PACK_SIGNATURE is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 228 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.eclipse.jgit.lib.DefaultTypedConfigGetter.getLong(Config, String, String, String, long) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 169-177 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.eclipse.jgit.lib.GpgSignature(byte[]) may expose internal representation by storing an externally mutable object into GpgSignature.signature | MALICIOUS_CODE | EI_EXPOSE_REP2 | 69 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.ObjectChecker.author is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 123 | Medium |
org.eclipse.jgit.lib.ObjectChecker.committer is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 126 | Medium |
org.eclipse.jgit.lib.ObjectChecker.encoding is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 129 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tagger is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 141 | Medium |
org.eclipse.jgit.lib.ObjectChecker.object should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 132 | Medium |
org.eclipse.jgit.lib.ObjectChecker.parent should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 120 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tag should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 138 | Medium |
org.eclipse.jgit.lib.ObjectChecker.tree should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 117 | Medium |
org.eclipse.jgit.lib.ObjectChecker.type should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 135 | Medium |
Switch statement found in org.eclipse.jgit.lib.ObjectChecker.checkNotWindowsDevice(byte[], int, int, AnyObjectId) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 989-1046 | 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 | 322 | 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 | 302 | 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 | 103 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
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 | 191 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.lib.RefDatabase.SEARCH_PATH should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 78 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Sequence of calls to java.util.concurrent.ConcurrentHashMap may not be atomic in org.eclipse.jgit.lib.RepositoryCache.openRepository(RepositoryCache$Key, boolean) | MT_CORRECTNESS | AT_OPERATION_SEQUENCE_ON_CONCURRENT_ABSTRACTION | 276 | Medium |
Synchronization performed on java.util.concurrent.ScheduledThreadPoolExecutor in org.eclipse.jgit.lib.RepositoryCache.configureEviction(RepositoryCacheConfig) | MT_CORRECTNESS | JLM_JSR166_UTILCONCURRENT_MONITORENTER | 250 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of gpgKey, which is known to be non-null in org.eclipse.jgit.lib.internal.BouncyCastleGpgSigner.canLocateSigningKey(String, PersonIdent, CredentialsProvider) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 104 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.eclipse.jgit.merge.Merger.monitor | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 108 | 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 | 1127 | 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 | 1117 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.eclipse.jgit.merge.ResolveMerger.checkout() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 417 | Medium |
Exceptional return value of java.io.File.mkdirs() ignored in org.eclipse.jgit.merge.ResolveMerger.writeMergedFile(TemporaryBuffer, Attributes) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 1013 | 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 | 175 | 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 | 105 | 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 | 720 | 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 | 374 | Medium |
org.eclipse.jgit.revwalk.RevCommit.getRawBuffer() may expose internal representation by returning RevCommit.buffer | MALICIOUS_CODE | EI_EXPOSE_REP | 390 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.revwalk.RevWalk$3.next() can't throw NoSuchElementException | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 1405-1407 | Medium |
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 | 199 | 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 | 225 | Medium |
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 |
---|---|---|---|---|
Inconsistent synchronization of org.eclipse.jgit.transport.JschConfigSessionFactory.config; locked 80% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 555 | Medium |
org.eclipse.jgit.transport.JschConfigSessionFactory.getSession(URIish, CredentialsProvider, FS, int) calls Thread.sleep() with a lock held | MT_CORRECTNESS | SWL_SLEEP_WITH_LOCK_HELD | 168 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of values, which is known to be non-null in org.eclipse.jgit.transport.OpenSshConfig$Host$1.getValues(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 337 | Medium |
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 |
---|---|---|---|---|
Comparison of String parameter using == or != in org.eclipse.jgit.transport.PacketLineIn.isDelimiter(String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 264 | High |
Comparison of String parameter using == or != in org.eclipse.jgit.transport.PacketLineIn.isEnd(String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 299 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.eclipse.jgit.transport.PushCertificateParser.receiveHeader(PushCertificateParser$StringReader, boolean) might ignore java.io.EOFException | BAD_PRACTICE | DE_MIGHT_IGNORE | 378 | Medium |
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 | 388 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Uninitialized read of hookOutRedirect in new org.eclipse.jgit.transport.Transport(Repository, URIish) | CORRECTNESS | UR_UNINIT_READ | 826 | Medium |
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 | 81 | 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 | 270 | Medium |
Switch statement found in org.eclipse.jgit.transport.WalkFetchConnection.markLocalObjComplete(RevObject) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 719-727 | 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 | 492 | Medium |
org.eclipse.jgit.treewalk.AbstractTreeIterator.zeroid is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 90 | 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 | 279 | 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 | 145 | 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 | 115 | 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 | 314 | Medium |
org.eclipse.jgit.treewalk.WorkingTreeIterator.idBuffer() may expose internal representation by returning WorkingTreeIterator.zeroid | MALICIOUS_CODE | EI_EXPOSE_REP | 345 | Medium |
Possible null pointer dereference in org.eclipse.jgit.treewalk.WorkingTreeIterator.isModified(DirCacheEntry, boolean, ObjectReader) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 1013 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
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 | 107-111 | 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 | 105-115 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Possible null pointer dereference in org.eclipse.jgit.util.FS$FileStoreAttributes.get(Path) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 281 | Medium |
Possible null pointer dereference in org.eclipse.jgit.util.FS$FileStoreAttributes.write(Path, String) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 443 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.eclipse.jgit.util.FS_POSIX.readUmask() | STYLE | REC_CATCH_EXCEPTION | 164 | 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 | 256 | 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 | 147-155 | 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 |
---|---|---|---|---|
Incorrect lazy initialization of static field org.eclipse.jgit.util.SystemReader.isMacOS in org.eclipse.jgit.util.SystemReader.isMacOS() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 445-447 | Medium |
Incorrect lazy initialization of static field org.eclipse.jgit.util.SystemReader.isWindows in org.eclipse.jgit.util.SystemReader.isWindows() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 432-434 | Medium |
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 | 447 | 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 | 434 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless condition: it's known that b == 10 at this point | STYLE | UC_USELESS_CONDITION | 132 | High |