SpotBugs 4.10.5

CHANGELOG

• https://github.com/spotbugs/spotbugs/blob/4.10.5/CHANGELOG.md

CHECKSUM

SpotBugs 4.10.4

CHANGELOG

Fixed

• Fix NN_NAKED_NOTIFY false negatives when a field read is stored in a local variable before notify() or notifyAll() (#3884)
• Fix ASE_ASSERTION_WITH_SIDE_EFFECT and ASE_ASSERTION_WITH_SIDE_EFFECT_METHOD false positives in every method analysed after a method that reads $assertionsDisabled without throwing an AssertionError (#3483)
• Fix INT_BAD_COMPARISON_WITH_SIGNED_BYTE false positive for meaningful comparisons of a signed byte with 127 (b < 127, b >= 127) (#4201)
• Fix EI_EXPOSE_REP false negative for public getters in anonymous classes (#4237)
• Fix missing class report for java.util.Collections$EmptyNavigableSet and java.util.Collections$EmptyNavigableMap when the result of Collections.emptySortedSet(), emptyNavigableSet(), emptySortedMap() or emptyNavigableMap() is stored (#4244)
• Fix URF_UNREAD_FIELD false negative for unread instance fields declared in enums (#4246)
• Stop publishing global dependency-management constraints to consumer POMs. (#4223)

CHECKSUM

SpotBugs 4.10.3

CHANGELOG

Fixed

• Fix LI_LAZY_INIT_STATIC false negative when the null guard is written in yoda-style (null == field) (#4144)
• Fix DC_DOUBLECHECK, NP_SYNC_AND_NULL_CHECK_FIELD and SP_SPIN_ON_FIELD false negatives when the null guard is written in yoda-style (null == field) (#4144)
• Fix message for UNS_UNSAFE_CALL bug pattern
• Restore CLI plugin loading by fixing DetectorFactoryCollection bootstrap ordering (#4191)
• Fix UWF_NULL_FIELD false negative for fields initialized with cast null values (#4034)
• Fix UMAC_UNCALLABLE_METHOD_OF_ANONYMOUS_CLASS false positive for methods reached only through method references (#4059)

Changed

• Ant FindBugsViewerTask: use default look and feel by default. (#4165)

Refactor

• Ant FindBugsViewerTask: extend AbstractFindBugsTask to reduce duplicate code. (#4165)

CHECKSUM

SpotBugs 4.10.2

CHANGELOG

Build

• Add release protection to ensure version released matches the tag and that snapshot has been removed. (#4156)
• Drop binary incompatible Saxon-HE back to 12.9 to keep java 11 compatibility. (#4159)
• Add binary check to the gradle build to ensure compatibility remains. (#4159)

CHECKSUM

SpotBugs 4.10.1

Note

SpotBugs 4.10.0 was superseded by 4.10.1 due to a release issue. Users should use 4.10.1. See the discussion below for additional details:

https://github.com/spotbugs/spotbugs/discussions/4155

CHANGELOG

Build

• 4.10.0 was not released due to a release process error (artifacts were built from a -SNAPSHOT version). 4.10.1 is the corrected release and contains the intended 4.10.0 contents.

CHECKSUM

Note: SpotBugs 4.10.0 has been superseded by 4.10.1 due to a release issue. Please use 4.10.1 instead. See https://github.com/spotbugs/spotbugs/discussions/4155

SpotBugs 4.10.0-SNAPSHOT

CHANGELOG

Refactor

• Move internal usage of 'javax.annotation.Nonnull' to 'jakarta.annotation.NonNull'. (#3858)
• Move internal usage of 'javax.annotation.Nullable' to 'jakarta.annotation.Nullable'. (#3861)
• Renamed methods from edu.umd.cs.findbugs.SwitchHandler to reflect that they return a PC, not an offset (#3869)
• Make the progress bar more visually appealing by adding some borders (#3896)
• Reuse DismantleBytecode.isIf introduced in (#3869)

Added

• Add partial support for org.jspecify.annotations.Nullable, org.jspecify.annotations.NonNull, org.jspecify.annotations.NullUnmarked and org.jspecify.annotations.NullMarked annotations. These are aliased to the closest existing SpotBugs nullness annotations. This is not a complete implementation of the JSpecify spec; scope-level semantics of @NullMarked and @NullUnmarked are not yet supported. (#3996)
• Recognize jakarta.annotation.Nonnull and jakarta.annotation.Nullable (#3780)
• Detect use of sun.misc.Unsafe and jdk.internal.misc.Unsafe (#3804)
• New bug type is introduced: NCR_NOT_PROPERLY_CHECKED_READ. Improper validation of the return value from the read() method in InputStream and Reader classes may result in an array not being fully filled. (#3766)
• New detector FindImproperSynchronization and introduced new bug types:
  • USO_UNSAFE_METHOD_SYNCHRONIZATION is reported when using synchronized methods with the class' accessible intrinsic lock,
  • USO_UNSAFE_STATIC_METHOD_SYNCHRONIZATION is reported when using static synchronized methods with the class' exposed intrinsic lock,
  • USO_UNSAFE_OBJECT_SYNCHRONIZATION is reported when the lock used for synchronization is visible from the outside,
  • USO_UNSAFE_ACCESSIBLE_OBJECT_SYNCHRONIZATION is reported when the lock used for synchronization is made accessible, with methods that update or return the lock, to the outside,
  • USO_UNSAFE_INHERITABLE_OBJECT_SYNCHRONIZATION is reported when the lock used for synchronization is can be altered by subclasses,
  • USO_UNSAFE_EXPOSED_OBJECT_SYNCHRONIZATION is reported when the lock used for synchronization is later exposed in the subclasses.
  • USBC_UNSAFE_SYNCHRONIZATION_WITH_BACKING_COLLECTION is reported when the backing collection of a lock is visible from the outside,
  • USBC_UNSAFE_SYNCHRONIZATION_WITH_ACCESSIBLE_BACKING_COLLECTION is reported when the backing collection of a lock is made accessible, with methods that update or return the lock, to the outside,
  • USBC_UNSAFE_SYNCHRONIZATION_WITH_INHERITABLE_BACKING_COLLECTION is reported when the backing collection of a lock can be altered by subclasses.
    (See SEI CERT rule LCK00-J and SEI CERT rule LCK04-J)
• New detector FindIncreasedAccessibilityOfMethods for new bug type IAOM_DO_NOT_INCREASE_METHOD_ACCESSIBILITY. This detector reports a bug if a class increases the accessibility of overridden or hidden methods. (See SEI CERT rule MET04-J)

Fixed

• Fix DM_STRING_TOSTRING false negative when toString() is chained before a method call (e.g., s.toString().toLowerCase()); multiple occurrences in the same method are now all reported (#3966)
• Stop exposing JUnit BOM as a transitive dependency to consumers (#3908)
• Fix incorrect bug counts and sizes when unioning reports (#3721)
• Classes containing only methods throwing UnsupportedOperationException with setter-like names are no longer considered as mutable (#1601)
• Enhanced SARIF output with full description sections - adding markdown is still an open issue (#2339)
• Added missing null check to MultipleInstantiationsOfSingletons detector (#3823)
• Fix invalid syntax in findbugsfilter.xsd (#3832)
• Fix CT_CONSTRUCTOR_THROW FP with public and private constructors (#3822)
• Fix tool name in usage info, (#3847)
• Fix the building of relative chains of ./././ in filenames in fbp files (#3852)
• Fix IllegalArgumentException initializing spotbugs when inside a fat jar on Java 25 (#3875)
• Do not report DM_DEFAULT_ENCODING for classes compiled with target >= 18 (#3866)
• Fix FS_BAD_DATE_FORMAT_FLAG_COMBO not suppressed by field-level annotation (#3838)
• Fix SF_SWITCH_FALLTHROUGH false positives (#3767)
• Recognize well-known exception-throwing utility methods when looking for exceptions thrown from constructors (#3821)
• Fix RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE false negative when non-null value is on the left side of null comparison (#3920)
• Fix IM_BAD_CHECK_FOR_ODD false negative when using Yoda-style comparison (1 == i % 2) (#3886)
• Fix PluginLoader.close() to continue closing all URLClassLoaders when one close operation fails, suppressing subsequent IOExceptions. (#3958)
• Fix broken bugDescriptions.html#TYPE links by restoring legacy bug type anchors in generated docs (#2113)
• Fix EI_EXPOSE_REP false negative in package-private classes that expose mutable state through methods overriding a public super-type (#4027)
• Fix errors in the 4.40 Eclipse in SpotBugs plugin project (#4052)

Removed

• Removed old deprecated methods:
  • assertPresentBugPattern(String, IMarker[]) protected method from de.tobject.findbugs.test.AbstractQuickfixTest deprecated since 2014,
  • setFontSizeHelper(Component[], float) protected method from edu.umd.cs.findbugs.gui2.FBFrame deprecated since 2010,
  • matchedPrefixes(String[], String) method from edu.umd.cs.findbugs.gui2.ViewFilter deprecated since 2010,
  • lookupFromUniqueId(String) method from edu.umd.cs.findbugs.BugCollection and edu.umd.cs.findbugs.SortedBugCollection deprecated since 2006,
  • create(BugReporter) method from edu.umd.cs.findbugs.DetectorFactory deprecated since 2008,
  • instantiateDetectorsInPass(BugReporter) method from edu.umd.cs.findbugs.plan.AnalysisPass deprecated since 2008,
  • getMessage(String) method from edu.umd.cs.findbugs.I18N deprecated since 2019,
  • getElementSignature() method from edu.umd.cs.findbugs.OpcodeStack.Item deprecated since 2008,
  • getFieldAnnotation() method from edu.umd.cs.findbugs.OpcodeStack.Item deprecated since 2006,
  • PluginLoader(URL) and PluginLoader(URL, ClassLoader) constructors from edu.umd.cs.findbugs.PluginLoader deprecated since 2010,
  • addSourceDir(String) method from edu.umd.cs.findbugs.Project deprecated since 2017,
  • getImplicitClasspathEntryList() method from edu.umd.cs.findbugs.Project deprecated since 2008,
  • write(String, boolean, String) method from edu.umd.cs.findbugs.Project deprecated since 2007,
  • getInteger(String, int) method from edu.umd.cs.findbugs.SystemProperties deprecated since 2010,
  • getId() method from edu.umd.cs.findbugs.ba.BasicBlock deprecated since 2010,
  • getArgument(InvokeInstruction, ConstantPoolGen, int, int) method from edu.umd.cs.findbugs.ba.Frame deprecated since 2010,
  • findDeclaredExceptions(InvokeInstruction, ConstantPoolGen) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2008,
  • findConcreteMethod(JavaClass, String, String) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • findXMethod(JavaClass, String, String, JavaClassAndMethodChooser) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • findXMethod(JavaClass[], String, String) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • findXMethod(JavaClass[], String, String, JavaClassAndMethodChooser) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • findMethod(JavaClass[], String, String) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • isConcrete(XMethod) method from edu.umd.cs.findbugs.ba.Hierarchy deprecated since 2007,
  • `d...

SpotBugs 4.9.8

CHANGELOG

Fixed

• Maven plugin reporting issue if -adjustPriority is not set (#3774)

CHECKSUM

SpotBugs 4.9.7

CHANGELOG

Fixed

• Fix Eclipse not always using latest preferences file state (#3740)
• Fix exception throw when singleton implementing Cloneable has no clone() method (#3727)
• Fix for missing -adjustPriority parameter in Eclipse preferences (#3687)
• Documentation of -adjustPriority parameter
• Functionality from DetectorFactory setEnabledButNonReporting(), getPriorityAdjustment() methods and BugInstance.adjustForDetector() is deprecated and moved to PriorityAdjuster (#3753)
• Improved FindNakedNotify to handle the case when the lock is loaded from a field (#3634)

Changed

• Support for fully qualified class names for detectors in -adjustPriority parameter
• Support for numerical and absolute priority adjustments
• Bump up Apache Commons BCEL to the version 6.11.0 (#3569)

Deprecated

• Add back and deprecate edu.umd.cs.findbugs.io.IO.close(InputStream) method. (#3756)

Build

• Allow our GA builds to work with JDK 25 (and drop support for JDK 24) (#3564)

CHECKSUM

SpotBugs 4.9.6

CHANGELOG

Fixed

• Fix exception throw when analyzing jakarta.servlet.http.HttpServletRequest method calls (#3711)

CHECKSUM

CHANGELOG

Fixed

• Fix for an error when a record method has the @SuppressFBWarnings annotation (#3622)
• Fix SF_SWITCH_FALLTHROUGH false positive when continuing a loop (#3617)
• CWO_CLOSED_WITHOUT_OPENED false positive (#3616)
• SF_SWITCH_NO_DEFAULT false positive fix for switch-arrow (#3645)
• Fix the issue with BCEL logging Duplicating value: ... (#3621)
• Add missing jakarta support for servlets / pre/post destroy (#3694)

Added

• Add 'java.nio.file.Path.of' to known types for path traversal checks (#3699)

Cleanup

• S1481: Unused local variables should be removed (#3654)
• Moved test libraries to jakarta namespace including switching off jsr305 where possible for jakarta.annotation (#3695)

CHECKSUM