The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
980 | 70 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.CamelExchangeException defines non-transient non-serializable instance field exchange | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.ExpectedBodyTypeException defines non-transient non-serializable instance field exchange | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.FailedToCreateProducerException defines non-transient non-serializable instance field endpoint | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
MessageVariableResolver.exchange not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to parameterExpressions in org.apache.camel.component.bean.BeanInfo.introspect(Class, Method) | STYLE | DLS_DEAD_LOCAL_STORE | 118 |
Dead store to content in org.apache.camel.component.bean.BeanInfo.createParameterUnmarshalExpressionForAnnotation(Class, Method, Class, Annotation) | STYLE | DLS_DEAD_LOCAL_STORE | 267 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.bean.BeanInvocation.getArgs() may expose internal representation by returning BeanInvocation.args | MALICIOUS_CODE | EI_EXPOSE_REP | 37 |
org.apache.camel.component.bean.BeanInvocation.BeanInvocation(Object, Method, Object[]) may expose internal representation by storing an externally mutable object into BeanInvocation.args | MALICIOUS_CODE | EI_EXPOSE_REP2 | 33 |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.component.bean.MethodInfo$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.bean.ParameterInfo.getAnnotations() may expose internal representation by returning ParameterInfo.annotations | MALICIOUS_CODE | EI_EXPOSE_REP | 40 |
org.apache.camel.component.bean.ParameterInfo.ParameterInfo(int, Class, Annotation[], Expression) may expose internal representation by storing an externally mutable object into ParameterInfo.annotations | MALICIOUS_CODE | EI_EXPOSE_REP2 | 35 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.file.FileEndpoint.getExcludedNamePostfixes() may expose internal representation by returning FileEndpoint.excludedNamePostfixes | MALICIOUS_CODE | EI_EXPOSE_REP | 201 |
org.apache.camel.component.file.FileEndpoint.getExcludedNamePrefixes() may expose internal representation by returning FileEndpoint.excludedNamePrefixes | MALICIOUS_CODE | EI_EXPOSE_REP | 189 |
org.apache.camel.component.file.FileEndpoint.setExcludedNamePostfixes(String[]) may expose internal representation by storing an externally mutable object into FileEndpoint.excludedNamePostfixes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 209 |
org.apache.camel.component.file.FileEndpoint.setExcludedNamePrefixes(String[]) may expose internal representation by storing an externally mutable object into FileEndpoint.excludedNamePrefixes | MALICIOUS_CODE | EI_EXPOSE_REP2 | 197 |
Bug | Category | Details | Line |
---|---|---|---|
JMXEndpoint.mbeanServer not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 47 |
Redundant nullcheck of level, which is known to be non-null in org.apache.camel.component.log.LogComponent.getLoggingLevel(Map) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 48 |
Bug | Category | Details | Line |
---|---|---|---|
SedaConsumer.thread not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.component.timer.TimerConsumer$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.timer.TimerEndpoint.getTime() may expose internal representation by returning TimerEndpoint.time | MALICIOUS_CODE | EI_EXPOSE_REP | 114 |
org.apache.camel.component.timer.TimerEndpoint.setTime(Date) may expose internal representation by storing an externally mutable object into TimerEndpoint.time | MALICIOUS_CODE | EI_EXPOSE_REP2 | 118 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.vm.VmComponent.queues isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 37 |
Bug | Category | Details | Line |
---|---|---|---|
Should org.apache.camel.converter.CollectionConverter.toArray(Collection) return a zero length array rather than null? | STYLE | PZLA_PREFER_ZERO_LENGTH_ARRAYS | 49 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.converter.NIOConverter.toByteBuffer(File) may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 61 |
org.apache.camel.converter.NIOConverter.toByteBuffer(File) ignores result of java.io.InputStream.read(byte[]) | BAD_PRACTICE | RR_NOT_CHECKED | 62 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.converter.jaxp.BytesSource.getData() may expose internal representation by returning BytesSource.data | MALICIOUS_CODE | EI_EXPOSE_REP | 53 |
org.apache.camel.converter.jaxp.BytesSource.BytesSource(byte[]) may expose internal representation by storing an externally mutable object into BytesSource.data | MALICIOUS_CODE | EI_EXPOSE_REP2 | 36 |
org.apache.camel.converter.jaxp.BytesSource.BytesSource(byte[], String) may expose internal representation by storing an externally mutable object into BytesSource.data | MALICIOUS_CODE | EI_EXPOSE_REP2 | 40 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.converter.jaxp.StringSource is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.converter.jaxp.XmlConverter.defaultCharset isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 65 |
Redundant nullcheck of source, which is known to be non-null in org.apache.camel.converter.jaxp.XmlConverter.toDOMSource(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 189 |
Exception is caught when Exception is not thrown in org.apache.camel.converter.jaxp.XmlConverter.toSAXSourceFromDOM(DOMSource) | STYLE | REC_CATCH_EXCEPTION | 361 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.impl.DefaultCamelContext implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Private method org.apache.camel.impl.DefaultExchange.safeCopy(Exchange, Message) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 97-105 |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.camel.impl.converter.DefaultTypeConverter.checkLoaded() | STYLE | REC_CATCH_EXCEPTION | 216 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.language.ExpressionEvaluationException defines non-transient non-serializable instance field exchange | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Class org.apache.camel.language.ExpressionEvaluationException defines non-transient non-serializable instance field expression | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.language.IllegalSyntaxException defines non-transient non-serializable instance field language | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Method call in org.apache.camel.management.CamelNamingStrategy.getObjectName(ManagedRoute) passes null for unconditionally dereferenced parameter of getEndpointId(Endpoint) | CORRECTNESS | NP_NULL_PARAM_DEREF | 123 |
Bug | Category | Details | Line |
---|---|---|---|
Confusing to have methods org.apache.camel.management.InstrumentationAgentImpl.getMBeanServer() and org.apache.camel.component.jmx.JMXComponent.getMbeanServer() | BAD_PRACTICE | NM_CONFUSING | 64 |
Confusing to have methods org.apache.camel.management.InstrumentationAgentImpl.setMBeanServer(MBeanServer) and org.apache.camel.component.jmx.JMXEndpoint.setMbeanServer(MBeanServer) | BAD_PRACTICE | NM_CONFUSING | 60-61 |
InstrumentationAgentImpl.server not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Private method org.apache.camel.management.ManagedService.isStarted() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 45 |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.model.AggregatorType$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unread field: org.apache.camel.model.ProcessorType.lastInterceptor | PERFORMANCE | URF_UNREAD_FIELD | 508 |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.model.ProcessorType$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.model.ResequencerType$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to endpoint in org.apache.camel.model.RouteType.addRoutes(Collection, FromType) | STYLE | DLS_DEAD_LOCAL_STORE | 158 |
Bug | Category | Details | Line |
---|---|---|---|
BatchProcessor.consumer not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.processor.Pipeline implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.processor.RedeliveryPolicy.randomNumberGenerator should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
org.apache.camel.processor.RedeliveryPolicy is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
The field org.apache.camel.processor.RedeliveryPolicy.randomNumberGenerator is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.processor.SendProcessor implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.processor.idempotent.NoMessageIdException defines non-transient non-serializable instance field exchange | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Class org.apache.camel.processor.idempotent.NoMessageIdException defines non-transient non-serializable instance field expression | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.util.ExpressionComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.util.ExpressionListComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.util.FactoryFinder.doFindFactoryProperies(String) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 170 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 201 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 76 |
Method org.apache.camel.util.IntrospectionSupport.extractProperties(Map, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 122 |
Method org.apache.camel.util.IntrospectionSupport.setProperties(Object, Map, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 101 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.util.ResolverUtil.loadImplementationsInJar(ResolverUtil$Test, String, File) may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 327 |
Bug | Category | Details | Line |
---|---|---|---|
Method org.apache.camel.util.URISupport.createQueryString(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 282 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.util.URISupport$CompositeData.getComponents() may expose internal representation by returning URISupport$CompositeData.components | MALICIOUS_CODE | EI_EXPOSE_REP | 44 |
URISupport$CompositeData.components not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
URISupport$CompositeData.parameters not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.camel.util.UuidGenerator.<static initializer>() | STYLE | REC_CATCH_EXCEPTION | 56 |