FindBugs Bug Detector Report
The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
Summary
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 1282 | 237 | 0 | 0 |
Files
org.apache.myfaces.application.ApplicationImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.application.ApplicationImpl.getELContextListeners() uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 368 |
| Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.internalCreateConverter(Class) | STYLE | REC_CATCH_EXCEPTION | 792 |
| Exception is caught when Exception is not thrown in org.apache.myfaces.application.ApplicationImpl.internalCreateConverter(Class) | STYLE | REC_CATCH_EXCEPTION | 803 |
org.apache.myfaces.application.DefaultViewHandlerSupport
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of mapping, which is known to be non-null in org.apache.myfaces.application.DefaultViewHandlerSupport.calculateViewId(FacesContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 53 |
org.apache.myfaces.application.NavigationHandlerImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Inconsistent synchronization of org.apache.myfaces.application.NavigationHandlerImpl._wildcardKeys; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 207 |
org.apache.myfaces.application.NavigationHandlerImpl$KeyComparator
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.application.NavigationHandlerImpl$KeyComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
org.apache.myfaces.application.jsp.JspStateManagerImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.application.jsp.JspStateManagerImpl.nextViewSequence(FacesContext) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 601 |
| Method org.apache.myfaces.application.jsp.JspStateManagerImpl.nextViewSequence(FacesContext) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 605 |
| Redundant nullcheck of component, which is known to be non-null in org.apache.myfaces.application.jsp.JspStateManagerImpl.getPathToComponent(UIComponent, StringBuffer) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 434 |
org.apache.myfaces.application.jsp.JspStateManagerImpl$SerializedViewCollection
| Bug | Category | Details | Line |
|---|---|---|---|
| The field org.apache.myfaces.application.jsp.JspStateManagerImpl$SerializedViewCollection._oldSerializedViews is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
org.apache.myfaces.application.jsp.JspStateManagerImpl$SerializedViewKey
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to PRIME in org.apache.myfaces.application.jsp.JspStateManagerImpl$SerializedViewKey.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 875 |
org.apache.myfaces.application.jsp.ViewResponseWrapper
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.application.jsp.ViewResponseWrapper.toString() may return null | BAD_PRACTICE | NP_TOSTRING_COULD_RETURN_NULL | 133 |
| ViewResponseWrapper._writer not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.config.FacesConfigValidator
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of ?, which is known to be non-null in org.apache.myfaces.config.FacesConfigValidator.validate(ExternalContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 53 |
| Redundant nullcheck of managedBeansMap, which is known to be non-null in org.apache.myfaces.config.FacesConfigValidator.validate(ExternalContext, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 49 |
org.apache.myfaces.config.FacesConfigurator
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to serialFactory in org.apache.myfaces.config.FacesConfigurator.handleSerialFactory() | STYLE | DLS_DEAD_LOCAL_STORE | 1030 |
| Load of known null value in org.apache.myfaces.config.FacesConfigurator.feedClassloaderConfigurations() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 514 |
| Load of known null value in org.apache.myfaces.config.FacesConfigurator.feedClassloaderConfigurations() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 517 |
| org.apache.myfaces.config.FacesConfigurator.feedMetaInfServicesFactories() may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 436 |
| Redundant nullcheck of stream which is known to be null in org.apache.myfaces.config.FacesConfigurator.feedClassloaderConfigurations() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 517 |
| Write to static field org.apache.myfaces.config.FacesConfigurator.lastUpdate from instance method org.apache.myfaces.config.FacesConfigurator.configure() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 284 |
org.apache.myfaces.config.annotation.DefaultLifecycleProviderFactory
| Bug | Category | Details | Line |
|---|---|---|---|
| Null pointer dereference of DefaultLifecycleProviderFactory.LIFECYCLE_PROVIDER_INSTANCE in org.apache.myfaces.config.annotation.DefaultLifecycleProviderFactory.getLifecycleProvider(ExternalContext) | CORRECTNESS | NP_ALWAYS_NULL | 67 |
| Write to static field org.apache.myfaces.config.annotation.DefaultLifecycleProviderFactory.LIFECYCLE_PROVIDER_INSTANCE from instance method org.apache.myfaces.config.annotation.DefaultLifecycleProviderFactory.release() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 73 |
org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupFieldResource(Context, Object, Field, String) invokes java.lang.reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 127 |
| org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupFieldResource(Context, Object, Field, String) invokes java.lang.reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 129 |
| org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupMethodResource(Context, Object, Method, String) invokes java.lang.reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 163 |
| org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider.lookupMethodResource(Context, Object, Method, String) invokes java.lang.reflect.Method.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 165 |
org.apache.myfaces.config.annotation.TomcatAnnotationLifecycleProvider
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.apache.myfaces.config.annotation.TomcatAnnotationLifecycleProvider.isAvailable() | STYLE | REC_CATCH_EXCEPTION | 65 |
| TomcatAnnotationLifecycleProvider.annotationProcessor not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.config.impl.FacesConfigEntityResolver
| Bug | Category | Details | Line |
|---|---|---|---|
| Method call in org.apache.myfaces.config.impl.FacesConfigEntityResolver.resolveEntity(String, String) passes null for unconditionally dereferenced parameter of java.util.jar.JarFile.getInputStream(ZipEntry) | CORRECTNESS | NP_NULL_PARAM_DEREF | 85 |
org.apache.myfaces.config.impl.digester.elements.FacesConfig
| Bug | Category | Details | Line |
|---|---|---|---|
| Confusing to have methods org.apache.myfaces.config.impl.digester.elements.FacesConfig.addNavigationRule(NavigationRule) and org.apache.myfaces.config.RuntimeConfig.addNavigationRule(NavigationRule) | BAD_PRACTICE | NM_CONFUSING | 66-67 |
org.apache.myfaces.config.impl.digester.elements.ManagedProperty
| Bug | Category | Details | Line |
|---|---|---|---|
| ManagedProperty._valueBinding not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.context.portlet.SessionMap
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.context.portlet.SessionMap.getAttribute(String) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 45 |
org.apache.myfaces.context.servlet.SessionMap
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.context.servlet.SessionMap.getAttribute(String) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 47 |
org.apache.myfaces.convert.ConverterUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.convert.ConverterUtils.convertToBoolean(Object) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 65 |
org.apache.myfaces.el.PropertyResolverImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.el.PropertyResolverImpl.getType(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 150 |
| Method org.apache.myfaces.el.PropertyResolverImpl.getValue(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 55 |
| Method org.apache.myfaces.el.PropertyResolverImpl.isReadOnly(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 116 |
| Method org.apache.myfaces.el.PropertyResolverImpl.setValue(Object, int, Object) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 98 |
org.apache.myfaces.el.ResolverForJSPInitializer
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.el.ResolverForJSPInitializer defines non-transient non-serializable instance field _resolverBuilder | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class org.apache.myfaces.el.ResolverForJSPInitializer defines non-transient non-serializable instance field _resolverForJSP | BAD_PRACTICE | SE_BAD_FIELD | Not available |
org.apache.myfaces.el.convert.ELResolverToPropertyResolver
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.el.convert.ELResolverToPropertyResolver.getType(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 97 |
| Method org.apache.myfaces.el.convert.ELResolverToPropertyResolver.getValue(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 72 |
| Method org.apache.myfaces.el.convert.ELResolverToPropertyResolver.isReadOnly(Object, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 46 |
| Method org.apache.myfaces.el.convert.ELResolverToPropertyResolver.setValue(Object, int, Object) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 133 |
org.apache.myfaces.el.convert.ValueBindingToValueExpression
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to PRIME in org.apache.myfaces.el.convert.ValueBindingToValueExpression.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 140 |
| Redundant nullcheck of context, which is known to be non-null in org.apache.myfaces.el.convert.ValueBindingToValueExpression.getFacesContext(ELContext) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 250 |
| Class org.apache.myfaces.el.convert.ValueBindingToValueExpression defines non-transient non-serializable instance field _valueBinding | BAD_PRACTICE | SE_BAD_FIELD | Not available |
org.apache.myfaces.el.convert.ValueExpressionToValueBinding
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to PRIME in org.apache.myfaces.el.convert.ValueExpressionToValueBinding.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 54 |
org.apache.myfaces.el.unified.resolver.FacesCompositeELResolver
| Bug | Category | Details | Line |
|---|---|---|---|
| Invocation of toString on an array in org.apache.myfaces.el.unified.resolver.FacesCompositeELResolver.FacesCompositeELResolver(FacesCompositeELResolver$Scope) | CORRECTNESS | DMI_INVOKING_TOSTRING_ON_ARRAY | 59 |
org.apache.myfaces.el.unified.resolver.GuiceResolver
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of property, which is known to be non-null in org.apache.myfaces.el.unified.resolver.GuiceResolver.getValue(ELContext, Object, Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 79 |
org.apache.myfaces.el.unified.resolver.ManagedBeanResolver
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.el.unified.resolver.ManagedBeanResolver.getFeatureDescriptors(ELContext, Object) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 257 |
org.apache.myfaces.lifecycle.RestoreViewExecutor
| Bug | Category | Details | Line |
|---|---|---|---|
| Private method org.apache.myfaces.lifecycle.RestoreViewExecutor.deriveViewId(FacesContext) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 135-174 |
org.apache.myfaces.renderkit.html.HtmlHiddenRenderer
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.renderkit.html.HtmlHiddenRenderer.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 67 |
org.apache.myfaces.renderkit.html.HtmlRenderKitImpl$1
| Bug | Category | Details | Line |
|---|---|---|---|
| The class org.apache.myfaces.renderkit.html.HtmlRenderKitImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
org.apache.myfaces.renderkit.html.HtmlResponseStateManager
| Bug | Category | Details | Line |
|---|---|---|---|
| Load of known null value in org.apache.myfaces.renderkit.html.HtmlResponseStateManager.getSavedState(FacesContext) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 192 |
| Should org.apache.myfaces.renderkit.html.HtmlResponseStateManager.getSavedState(FacesContext) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 181 |
org.apache.myfaces.shared_impl.renderkit.RendererUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.shared_impl.renderkit.RendererUtils.getViewSequence(FacesContext) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 898 |
| Null pointer dereference of content in org.apache.myfaces.shared_impl.renderkit.RendererUtils.loadResourceFile(FacesContext, String) | CORRECTNESS | NP_ALWAYS_NULL | 1069 |
| Possible null pointer dereference of content in org.apache.myfaces.shared_impl.renderkit.RendererUtils.loadResourceFile(FacesContext, String) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 1088 |
org.apache.myfaces.shared_impl.renderkit.html.HTML
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.shared_impl.renderkit.html.HTML.LABEL_PASSTHROUGH_ATTRIBUTES is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 325 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.ANCHOR_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 186 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.ANCHOR_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 199 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.ANCHOR_PASSTHROUGH_ATTRIBUTES_WITHOUT_ONCLICK_WITHOUT_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 207 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.ANCHOR_PASSTHROUGH_ATTRIBUTES_WITHOUT_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 203 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.BUTTON_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 298 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.BUTTON_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 303 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.BUTTON_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 307 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 123 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_EVENT_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 80 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_EVENT_ATTRIBUTES_WITHOUT_ONFOCUS should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 88 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_PASSTROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 150 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_PASSTROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 170 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_PASSTROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONFOCUS_AND_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 165 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_PASSTROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 155 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_PASSTROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 134 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_PASSTROUGH_ATTRIBUTES_WITHOUT_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 142 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_PASSTROUGH_ATTRIBUTES_WITHOUT_ONCLICK_WITHOUT_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 146 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_PASSTROUGH_ATTRIBUTES_WITHOUT_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 138 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.EVENT_HANDLER_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 72 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.EVENT_HANDLER_ATTRIBUTES_WITHOUT_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 58 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.FORM_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 217 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.FORM_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 226 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.IMG_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 241 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.IMG_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 254 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 265 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_FILE_UPLOAD_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 376 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 273 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 277 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.LABEL_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 318 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.MESSAGE_PASSTHROUGH_ATTRIBUTES_WITHOUT_TITLE_STYLE_AND_STYLE_CLASS should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 393 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.SELECT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 333 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.SELECT_TABLE_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 400 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.TABLE_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 343 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.TABLE_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 354 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.TEXTAREA_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 363 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.TEXTAREA_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 370 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.UL_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 404 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.UNIVERSAL_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 110 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.UNIVERSAL_ATTRIBUTES_WITHOUT_STYLE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 102 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 128 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.COMMON_FIELD_PASSTROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONFOCUS should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 160 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_FILE_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 380 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.INPUT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONFOCUS_AND_ONCLICK should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 282 |
| org.apache.myfaces.shared_impl.renderkit.html.HTML.UL_PASSTHROUGH_ATTRIBUTES should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 408 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlButtonRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to nestingForm in org.apache.myfaces.shared_impl.renderkit.html.HtmlButtonRendererBase.buildOnClick(UIComponent, FacesContext, ResponseWriter) | STYLE | DLS_DEAD_LOCAL_STORE | 197 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlCheckboxRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.model.SelectItem to javax.faces.model.SelectItemGroup in org.apache.myfaces.shared_impl.renderkit.html.HtmlCheckboxRendererBase.renderGroupOrItemCheckbox(FacesContext, UIComponent, SelectItem, boolean, Set, Converter, boolean, Integer) | STYLE | BC_UNCONFIRMED_CAST | 176 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlCheckboxRendererBase.renderCheckboxList(FacesContext, UISelectMany) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 121 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlCheckboxRendererBase.renderGroupOrItemCheckbox(FacesContext, UIComponent, SelectItem, boolean, Set, Converter, boolean, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 181 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlFormRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIForm in org.apache.myfaces.shared_impl.renderkit.html.HtmlFormRendererBase.decode(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 177 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIForm in org.apache.myfaces.shared_impl.renderkit.html.HtmlFormRendererBase.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 59 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlGroupRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.html.HtmlPanelGroup in org.apache.myfaces.shared_impl.renderkit.html.HtmlGroupRendererBase.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 65 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlLinkRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Private method org.apache.myfaces.shared_impl.renderkit.html.HtmlLinkRendererBase.renderLinkParameter(String, Object, StringBuffer, String, UIComponent) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 477-488 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UISelectOne in org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 55 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UISelectOne in org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 303 |
| Unchecked/unconfirmed cast from javax.faces.model.SelectItem to javax.faces.model.SelectItemGroup in org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase.renderGroupOrItemRadio(FacesContext, UIComponent, SelectItem, Object, Converter, boolean, Integer) | STYLE | BC_UNCONFIRMED_CAST | 166 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase.encodeEnd(FacesContext, UIComponent) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 99 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlRadioRendererBase.renderGroupOrItemRadio(FacesContext, UIComponent, SelectItem, Object, Converter, boolean, Integer) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 170 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.findUIOutputConverterFailSafe(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 393 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UISelectMany in org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.findUISelectManyConverterFailsafe(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 380 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UISelectMany in org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.getSubmittedOrSelectedValuesAsSet(boolean, UIComponent, FacesContext, Converter) | STYLE | BC_UNCONFIRMED_CAST | 350 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UISelectOne in org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.getSubmittedOrSelectedValuesAsSet(boolean, UIComponent, FacesContext, Converter) | STYLE | BC_UNCONFIRMED_CAST | 357 |
| org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.DEFAULT_CHAR_ENCODING isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 1519 |
| Method call in org.apache.myfaces.shared_impl.renderkit.html.HtmlRendererUtils.renderDisplayValueOnlyForSelects(FacesContext, UIComponent) passes null for unconditionally dereferenced parameter of renderSelectOptionsAsText(FacesContext, UIComponent, Converter, Set, List, boolean) | CORRECTNESS | NP_NULL_PARAM_DEREF | 705 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlResponseWriterImpl
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 117 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 185 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 259 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 355 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlSecretRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.shared_impl.renderkit.html.HtmlSecretRendererBase.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 127 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIData in org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.encodeBegin(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 109 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIData in org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.encodeChildren(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 137 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIData in org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.encodeEnd(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 566 |
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIData in org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.encodeInnerHtml(FacesContext, UIComponent) | STYLE | BC_UNCONFIRMED_CAST | 239 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.renderColumnFooterCell(FacesContext, ResponseWriter, UIComponent, UIComponent, String, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 979 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.renderColumnHeaderCell(FacesContext, ResponseWriter, UIComponent, UIComponent, String, int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 925 |
| Method org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.renderTableHeaderOrFooterRow(FacesContext, ResponseWriter, UIComponent, UIComponent, String, String, int, boolean) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 781 |
| Redundant nullcheck of elemName, which is known to be non-null in org.apache.myfaces.shared_impl.renderkit.html.HtmlTableRendererBase.renderFacet(FacesContext, ResponseWriter, UIComponent, boolean) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 650 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlTextRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.shared_impl.renderkit.html.HtmlTextRendererBase.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 219 |
org.apache.myfaces.shared_impl.renderkit.html.HtmlTextareaRendererBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from javax.faces.component.UIComponent to javax.faces.component.UIOutput in org.apache.myfaces.shared_impl.renderkit.html.HtmlTextareaRendererBase.getConvertedValue(FacesContext, UIComponent, Object) | STYLE | BC_UNCONFIRMED_CAST | 94 |
org.apache.myfaces.shared_impl.renderkit.html.util.JavascriptUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 155 |
org.apache.myfaces.shared_impl.taglib.UIComponentBodyELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Private method org.apache.myfaces.shared_impl.taglib.UIComponentBodyELTagBase.setConverterProperty(UIComponent, String) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 209-210 |
org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 80 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 82 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 81 |
org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 80 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 82 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlDataTableTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 81 |
org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 76 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 75 |
org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 76 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputSecretTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 75 |
org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 69 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 71 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 70 |
org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 69 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 71 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlInputTextareaTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 70 |
org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 75 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 77 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 76 |
org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 75 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 77 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlPanelGridTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 76 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 67 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 69 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 68 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 67 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 69 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectBooleanCheckboxTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 68 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 70 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 72 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 71 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 70 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 72 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectListboxTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 71 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 79 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 81 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 80 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 79 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 81 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectManyCheckboxTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 80 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 76 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 75 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 76 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectMenuTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 75 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioELTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioELTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 72 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioELTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioELTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 73 |
org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioTagBase
| Bug | Category | Details | Line |
|---|---|---|---|
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioTagBase._datafld | PERFORMANCE | URF_UNREAD_FIELD | 72 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioTagBase._dataformatas | PERFORMANCE | URF_UNREAD_FIELD | 74 |
| Unread field: org.apache.myfaces.shared_impl.taglib.html.HtmlSelectOneRadioTagBase._datasrc | PERFORMANCE | URF_UNREAD_FIELD | 73 |
org.apache.myfaces.shared_impl.test.ClassElementHandler
| Bug | Category | Details | Line |
|---|---|---|---|
| ClassElementHandler.buffer not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.shared_impl.trace.TracingIterator
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.shared_impl.trace.TracingIterator.next() can't throw NoSuchElement exception | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 69 |
org.apache.myfaces.shared_impl.util.ExceptionUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.util.ExceptionUtils.getExceptions(Throwable) | STYLE | REC_CATCH_EXCEPTION | 59 |
org.apache.myfaces.shared_impl.util.LocaleUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.util.LocaleUtils.converterTagLocaleFromString(String) | STYLE | REC_CATCH_EXCEPTION | 136 |
org.apache.myfaces.shared_impl.util.MessageUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Load of known null value in org.apache.myfaces.shared_impl.util.MessageUtils.getMessage(FacesContext, String, Object[]) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 372 |
| Redundant nullcheck of message, which is known to be non-null in org.apache.myfaces.shared_impl.util.MessageUtils.getMessage(FacesContext, String, Object[]) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 374 |
org.apache.myfaces.shared_impl.util.SelectItemsIterator
| Bug | Category | Details | Line |
|---|---|---|---|
| SelectItemsIterator._currentUISelectItems not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.shared_impl.util.StateUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store of null to baos in org.apache.myfaces.shared_impl.util.StateUtils.compress(byte[]) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 215 |
| Dead store of null to gzip in org.apache.myfaces.shared_impl.util.StateUtils.compress(byte[]) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 214 |
| Dead store of null to bais in org.apache.myfaces.shared_impl.util.StateUtils.decompress(byte[]) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 279 |
| Dead store of null to baos in org.apache.myfaces.shared_impl.util.StateUtils.decompress(byte[]) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 278 |
| Dead store of null to gis in org.apache.myfaces.shared_impl.util.StateUtils.decompress(byte[]) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 280 |
| Dead store of null to outputStream in org.apache.myfaces.shared_impl.util.StateUtils.getAsByteArray(Object, ExternalContext) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 189 |
| Dead store of null to writer in org.apache.myfaces.shared_impl.util.StateUtils.getAsByteArray(Object, ExternalContext) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 188 |
| Dead store of null to input in org.apache.myfaces.shared_impl.util.StateUtils.getAsObject(byte[], ExternalContext) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 320 |
| Dead store of null to s in org.apache.myfaces.shared_impl.util.StateUtils.getAsObject(byte[], ExternalContext) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 319 |
| Random object created and used only once in org.apache.myfaces.shared_impl.util.StateUtils.findSecret(ServletContext) | BAD_PRACTICE | DMI_RANDOM_USED_ONLY_ONCE | 502 |
| Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.util.StateUtils.getAsObject(byte[], ExternalContext) | STYLE | REC_CATCH_EXCEPTION | 323 |
| Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.util.StateUtils.symmetric(byte[], SecretKey, String, String, byte[], int) | STYLE | REC_CATCH_EXCEPTION | 371 |
org.apache.myfaces.shared_impl.util.StringUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.shared_impl.util.StringUtils.trim(String[]) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 685 |
org.apache.myfaces.shared_impl.util.servlet.SourceCodeServlet
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.shared_impl.util.servlet.SourceCodeServlet is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.shared_impl.webapp.webxml.WebXmlParser
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.apache.myfaces.shared_impl.webapp.webxml.WebXmlParser.parse() | STYLE | REC_CATCH_EXCEPTION | 112 |
| WebXmlParser._webXml not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.taglib.core.DelegateValueChangeListener
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to className in org.apache.myfaces.taglib.core.DelegateValueChangeListener._createValueChangeListener() | STYLE | DLS_DEAD_LOCAL_STORE | 126 |
| Load of known null value in org.apache.myfaces.taglib.core.DelegateValueChangeListener._createValueChangeListener() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 139 |
org.apache.myfaces.taglib.core.GenericListenerTag
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of org.apache.myfaces.taglib.core.GenericListenerTag._type, which is known to be non-null in org.apache.myfaces.taglib.core.GenericListenerTag.doStartTag() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 88 |
| Redundant nullcheck of org.apache.myfaces.taglib.core.GenericListenerTag._type, which is known to be non-null in org.apache.myfaces.taglib.core.GenericListenerTag.doStartTag() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 104 |
org.apache.myfaces.taglib.core.LoadBundleTag$BundleMap
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of ?, which is known to be non-null in org.apache.myfaces.taglib.core.LoadBundleTag$BundleMap.containsKey(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 151 |
org.apache.myfaces.taglib.core.PhaseListenerTag
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.taglib.core.PhaseListenerTag defines non-transient non-serializable instance field log | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| org.apache.myfaces.taglib.core.PhaseListenerTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.taglib.core.PhaseListenerTag$BindingPhaseListener
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.apache.myfaces.taglib.core.PhaseListenerTag$BindingPhaseListener defines non-transient non-serializable instance field log | BAD_PRACTICE | SE_BAD_FIELD | Not available |
org.apache.myfaces.taglib.core.SetPropertyActionListenerTag
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.taglib.core.SetPropertyActionListenerTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
org.apache.myfaces.taglib.core.ValidatorImplTag
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.taglib.core.ValidatorImplTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
| ValidatorImplTag._binding not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
| ValidatorImplTag._validatorId not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.taglib.core.ValidatorTag
| Bug | Category | Details | Line |
|---|---|---|---|
| org.apache.myfaces.taglib.core.ValidatorTag is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
| ValidatorTag._binding not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
| ValidatorTag._validatorId not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
org.apache.myfaces.taglib.html.HtmlFormTag
| Bug | Category | Details | Line |
|---|---|---|---|
| Confusing to have methods org.apache.myfaces.taglib.html.HtmlFormTag.setAcceptcharset(ValueExpression) and org.apache.myfaces.shared_impl.taglib.html.HtmlFormELTagBase.setAcceptCharset(ValueExpression) | BAD_PRACTICE | NM_CONFUSING | 159-160 |
org.apache.myfaces.util.AbstractAttributeMap$EntrySetEntry
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to PRIME in org.apache.myfaces.util.AbstractAttributeMap$EntrySetEntry.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 379 |
org.apache.myfaces.util.DebugUtils
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.apache.myfaces.util.DebugUtils.printAttribute(PrintStream, String, Object) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 357 |
org.apache.myfaces.webapp.Jsp20FacesInitializer$ErrorExpressionFactory
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.apache.myfaces.webapp.Jsp20FacesInitializer$ErrorExpressionFactory be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |