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 |
---|---|---|---|
1781 | 186 | 0 | 16 |
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 |
---|---|---|---|
The class org.apache.camel.bam.ActivityBuilder$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.bam.ActivityBuilder$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.bam.model.ActivityState.getTimeExpected() may expose internal representation by returning ActivityState.timeExpected | MALICIOUS_CODE | EI_EXPOSE_REP | 127 |
org.apache.camel.bam.model.ActivityState.getTimeOverdue() may expose internal representation by returning ActivityState.timeOverdue | MALICIOUS_CODE | EI_EXPOSE_REP | 136 |
org.apache.camel.bam.model.ActivityState.setTimeExpected(Date) may expose internal representation by storing an externally mutable object into ActivityState.timeExpected | MALICIOUS_CODE | EI_EXPOSE_REP2 | 131 |
org.apache.camel.bam.model.ActivityState.setTimeOverdue(Date) may expose internal representation by storing an externally mutable object into ActivityState.timeOverdue | MALICIOUS_CODE | EI_EXPOSE_REP2 | 140 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.bam.model.ProcessInstance.getTimeCompleted() may expose internal representation by returning ProcessInstance.timeCompleted | MALICIOUS_CODE | EI_EXPOSE_REP | 99 |
org.apache.camel.bam.model.ProcessInstance.getTimeStarted() may expose internal representation by returning ProcessInstance.timeStarted | MALICIOUS_CODE | EI_EXPOSE_REP | 90 |
org.apache.camel.bam.model.ProcessInstance.setTimeCompleted(Date) may expose internal representation by storing an externally mutable object into ProcessInstance.timeCompleted | MALICIOUS_CODE | EI_EXPOSE_REP2 | 103 |
org.apache.camel.bam.model.ProcessInstance.setTimeStarted(Date) may expose internal representation by storing an externally mutable object into ProcessInstance.timeStarted | MALICIOUS_CODE | EI_EXPOSE_REP2 | 94 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.bam.model.TemporalEntity.getTimeCompleted() may expose internal representation by returning TemporalEntity.timeCompleted | MALICIOUS_CODE | EI_EXPOSE_REP | 53 |
org.apache.camel.bam.model.TemporalEntity.getTimeStarted() may expose internal representation by returning TemporalEntity.timeStarted | MALICIOUS_CODE | EI_EXPOSE_REP | 44 |
org.apache.camel.bam.model.TemporalEntity.setTimeCompleted(Date) may expose internal representation by storing an externally mutable object into TemporalEntity.timeCompleted | MALICIOUS_CODE | EI_EXPOSE_REP2 | 57 |
org.apache.camel.bam.model.TemporalEntity.setTimeStarted(Date) may expose internal representation by storing an externally mutable object into TemporalEntity.timeStarted | MALICIOUS_CODE | EI_EXPOSE_REP2 | 48 |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.camel.bam.processor.ActivityMonitorEngine.run() | STYLE | REC_CATCH_EXCEPTION | 97 |
Unwritten field: org.apache.camel.bam.processor.ActivityMonitorEngine.escalateLevel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 44 |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.bam.processor.NoCorrelationKeyException defines non-transient non-serializable instance field processor | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unread field: org.apache.camel.bam.rules.ActivityRules.activityDefinition | PERFORMANCE | URF_UNREAD_FIELD | 80 |
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 |
---|---|---|---|
Dead store to u in org.apache.camel.component.cxf.CxfComponent.createEndpoint(String, String, Map) | STYLE | DLS_DEAD_LOCAL_STORE | 50 |
Bug | Category | Details | Line |
---|---|---|---|
CxfInvokeProducer.client not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Dead store to response in org.apache.camel.component.cxf.CxfProducer.process(CxfExchange) | STYLE | DLS_DEAD_LOCAL_STORE | 71 |
CxfProducer.conduit not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Should org.apache.camel.component.cxf.CxfProducer$ResultFuture be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Load of known null value in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream.handleResponse() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 215 |
Load of known null value in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream.handleResponse() | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 225 |
A known null value is checked to see if it is an instance of java.lang.String in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream.handleResponse() | CORRECTNESS | NP_NULL_INSTANCEOF | 219 |
Non-virtual method call in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream.handleResponse() passes null for unconditionally dereferenced parameter of java.io.ByteArrayInputStream.ByteArrayInputStream(byte[]) | CORRECTNESS | NP_NULL_PARAM_DEREF_NONVIRTUAL | 225 |
Bug | Category | Details | Line |
---|---|---|---|
Redundant nullcheck of correlationID which is known to be null in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream$1.process(Exchange) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 170 |
Redundant nullcheck of id which is known to be null in org.apache.camel.component.cxf.transport.CamelConduit$CamelOutputStream$1.process(Exchange) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 162 |
Bug | Category | Details | Line |
---|---|---|---|
Should org.apache.camel.component.cxf.transport.CamelConduit$DecoupledDestination be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unread field: org.apache.camel.component.cxf.transport.CamelDestination.endpoint | PERFORMANCE | URF_UNREAD_FIELD | 86 |
Unread field: org.apache.camel.component.cxf.transport.CamelDestination.conduitInitiator | PERFORMANCE | URF_UNREAD_FIELD | 64 |
Unwritten field: org.apache.camel.component.cxf.transport.CamelDestination.camelUri | CORRECTNESS | UWF_UNWRITTEN_FIELD | 86 |
Bug | Category | Details | Line |
---|---|---|---|
Unused field: org.apache.camel.component.cxf.transport.CamelDestination$CamelOutputStream.replyTo | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Unused field: org.apache.camel.component.cxf.transport.CamelDestination$CamelOutputStream.sender | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.cxf.transport.CamelTransportBase.close() might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 60 |
Unread field: org.apache.camel.component.cxf.transport.CamelTransportBase.camelContext | PERFORMANCE | URF_UNREAD_FIELD | 38 |
Unwritten field: org.apache.camel.component.cxf.transport.CamelTransportBase.replyDestination | CORRECTNESS | UWF_UNWRITTEN_FIELD | 49 |
Bug | Category | Details | Line |
---|---|---|---|
Method call in org.apache.camel.component.cxf.transport.CamelTransportFactory.getConduit(EndpointInfo) passes null for unconditionally dereferenced parameter of getConduit(EndpointInfo, EndpointReferenceType) | CORRECTNESS | NP_NULL_PARAM_DEREF_ALL_TARGETS_DANGEROUS | 68 |
CamelTransportFactory.bus not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Class org.apache.camel.component.event.CamelEvent defines non-transient non-serializable instance field exchange | BAD_PRACTICE | SE_BAD_FIELD | Not available |
org.apache.camel.component.event.CamelEvent is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
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 |
---|---|---|---|
Unread field: org.apache.camel.component.file.remote.RemoteFileConsumer.endpoint | PERFORMANCE | URF_UNREAD_FIELD | 29 |
Bug | Category | Details | Line |
---|---|---|---|
Result of integer multiplication cast to long in org.apache.camel.component.file.remote.SftpConsumer.pollFile(ChannelSftp$LsEntry) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 78 |
Method org.apache.camel.component.file.remote.SftpConsumer.pollDirectory(String) uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 60 |
Bug | Category | Details | Line |
---|---|---|---|
Invocation of toString on an array in org.apache.camel.component.irc.IrcConfiguration.toString() | CORRECTNESS | DMI_INVOKING_TOSTRING_ON_ARRAY | 219 |
org.apache.camel.component.irc.IrcConfiguration.getPorts() may expose internal representation by returning IrcConfiguration.ports | MALICIOUS_CODE | EI_EXPOSE_REP | 123 |
org.apache.camel.component.irc.IrcConfiguration.setPorts(int[]) may expose internal representation by storing an externally mutable object into IrcConfiguration.ports | MALICIOUS_CODE | EI_EXPOSE_REP2 | 127 |
Bug | Category | Details | Line |
---|---|---|---|
Unwritten field: org.apache.camel.component.irc.IrcConsumer.listener | CORRECTNESS | UWF_UNWRITTEN_FIELD | 47 |
Bug | Category | Details | Line |
---|---|---|---|
Unread field: org.apache.camel.component.irc.IrcConsumer$FilteredIRCEventAdapter.target | PERFORMANCE | URF_UNREAD_FIELD | 71 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.component.irc.IrcProducer.COMMANDS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 29 |
Bug | Category | Details | Line |
---|---|---|---|
Confusing to have methods org.apache.camel.component.jms.EndpointMessageListener.onMessage(Message) and org.apache.camel.component.cxf.CxfProducer$ResultFuture.onMessage(Message) | BAD_PRACTICE | NM_CONFUSING | 49-58 |
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 |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.camel.component.mail.MailBinding.populateMailMessage(MailEndpoint, MimeMessage, Exchange) | STYLE | REC_CATCH_EXCEPTION | 54 |
Bug | Category | Details | Line |
---|---|---|---|
Confusing to have methods org.apache.camel.component.mail.MailMessage.setMessage(Message) and org.apache.camel.component.cxf.CxfMessage.setMessage(Message) | BAD_PRACTICE | NM_CONFUSING | 74-75 |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.component.mina.MinaProducer$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Non-virtual method call in org.apache.camel.component.quartz.QuartzEndpoint.createExchange() passes null for unconditionally dereferenced parameter of QuartzExchange.QuartzExchange(CamelContext, JobExecutionContext) | CORRECTNESS | NP_NULL_PARAM_DEREF_NONVIRTUAL | 121 |
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 |
---|---|---|---|
Confusing to have methods org.apache.camel.component.validator.jing.JingValidator.setSchemaFactory(SchemaFactory) and org.apache.camel.processor.validation.ValidatingProcessor.setSchemaFactory(SchemaFactory) | BAD_PRACTICE | NM_CONFUSING | 139-140 |
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 |
---|---|---|---|
XmppGroupChatProducer.chat not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
XmppPrivateChatProducer.chat not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
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 |
---|---|---|---|
Confusing to have methods org.apache.camel.processor.validation.ValidatingProcessor.setSchema(Schema) and org.apache.camel.component.validator.jing.JingValidator.setSchema(Schema) | BAD_PRACTICE | NM_CONFUSING | 124-125 |
Bug | Category | Details | Line |
---|---|---|---|
CamelBeanPostProcessor.applicationContext not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
CamelBeanPostProcessor.camelContext not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.spring.CamelBeanPostProcessor$3 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.spring.CamelContextFactoryBean.getPackages() may expose internal representation by returning CamelContextFactoryBean.packages | MALICIOUS_CODE | EI_EXPOSE_REP | 179 |
org.apache.camel.spring.CamelContextFactoryBean.setPackages(String[]) may expose internal representation by storing an externally mutable object into CamelContextFactoryBean.packages | MALICIOUS_CODE | EI_EXPOSE_REP2 | 191 |
Unused field: org.apache.camel.spring.CamelContextFactoryBean.beans | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Unused field: org.apache.camel.spring.CamelContextFactoryBean.endpoints | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
CamelContextFactoryBean.applicationContext not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unused field: org.apache.camel.spring.CamelProxyFactoryType.serviceInterface | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Unused field: org.apache.camel.spring.CamelProxyFactoryType.serviceUrl | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unused field: org.apache.camel.spring.CamelServiceExporterType.serviceInterface | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Unused field: org.apache.camel.spring.CamelServiceExporterType.serviceRef | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Unused field: org.apache.camel.spring.CamelServiceExporterType.uri | PERFORMANCE | UUF_UNUSED_FIELD | Not available |
Bug | Category | Details | Line |
---|---|---|---|
EndpointFactoryBean.context not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Unread field: org.apache.camel.spring.Main$ParameterOption.parameterName | PERFORMANCE | URF_UNREAD_FIELD | 183 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.spring.RouteBuilderFinder.getPackages() may expose internal representation by returning RouteBuilderFinder.packages | MALICIOUS_CODE | EI_EXPOSE_REP | 47 |
org.apache.camel.spring.RouteBuilderFinder.RouteBuilderFinder(SpringCamelContext, String[]) may expose internal representation by storing an externally mutable object into RouteBuilderFinder.packages | MALICIOUS_CODE | EI_EXPOSE_REP2 | 43 |
Bug | Category | Details | Line |
---|---|---|---|
The class org.apache.camel.spring.handler.CamelNamespaceHandler$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
Bug | Category | Details | Line |
---|---|---|---|
SendBeforeInterceptor.invocationHandler not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.spring.util.MainRunner.getArgs() may expose internal representation by returning MainRunner.args | MALICIOUS_CODE | EI_EXPOSE_REP | 80 |
org.apache.camel.spring.util.MainRunner.setArgs(String[]) may expose internal representation by storing an externally mutable object into MainRunner.args | MALICIOUS_CODE | EI_EXPOSE_REP2 | 84 |
MainRunner.main not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.camel.spring.util.ReflectionUtils.setField(Field, Object, Object) invokes java.lang.reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 49 |
org.apache.camel.spring.util.ReflectionUtils.setField(Field, Object, Object) invokes java.lang.reflect.Field.setAccessible(boolean), which should be invoked from within a doPrivileged block | BAD_PRACTICE | DP_DO_INSIDE_DO_PRIVILEGED | 53 |
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 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.LogFactory.nullClassLoaderFactory should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 300 |
org.apache.commons.logging.LogFactory.factories isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 284 |
org.apache.commons.logging.LogFactory.getFactory() may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 511 |
org.apache.commons.logging.LogFactory.getFactory() may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 514 |
Exception is caught when Exception is not thrown in org.apache.commons.logging.LogFactory.createFactory(String, ClassLoader) | STYLE | REC_CATCH_EXCEPTION | 1192 |
Exception is caught when Exception is not thrown in org.apache.commons.logging.LogFactory.getFactory() | STYLE | REC_CATCH_EXCEPTION | 539 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.LogSource.logImplctor should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 70 |
org.apache.commons.logging.LogSource.logs isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 61 |
org.apache.commons.logging.LogSource.jdk14IsAvailable isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 67 |
org.apache.commons.logging.LogSource.log4jIsAvailable isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 64 |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.commons.logging.impl.Jdk13LumberjackLogger.getClassAndMethod() | STYLE | REC_CATCH_EXCEPTION | 131 |
org.apache.commons.logging.impl.Jdk13LumberjackLogger is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
The field org.apache.commons.logging.impl.Jdk13LumberjackLogger.logger is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.impl.Jdk14Logger is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
The field org.apache.commons.logging.impl.Jdk14Logger.logger is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in org.apache.commons.logging.impl.Log4JLogger.<static initializer>() | STYLE | REC_CATCH_EXCEPTION | 90 |
org.apache.commons.logging.impl.Log4JLogger is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
The field org.apache.commons.logging.impl.Log4JLogger.logger is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Redundant nullcheck of specifiedLogClassName, which is known to be non-null in org.apache.commons.logging.impl.LogFactoryImpl.discoverLogImplementation(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 792 |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.impl.LogKitLogger is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
The field org.apache.commons.logging.impl.LogKitLogger.logger is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.impl.NoOpLog is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
org.apache.commons.logging.impl.SimpleLog.dateFormatter isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 103 |
org.apache.commons.logging.impl.SimpleLog.dateTimeFormat isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 101 |
org.apache.commons.logging.impl.SimpleLog.showDateTime isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 99 |
org.apache.commons.logging.impl.SimpleLog.showLogName isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 92 |
org.apache.commons.logging.impl.SimpleLog.showShortName isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 97 |
Redundant nullcheck of classLoader which is known to be null in org.apache.commons.logging.impl.SimpleLog.getContextClassLoader() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 640 |
org.apache.commons.logging.impl.SimpleLog is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Dead store of null to result in org.apache.commons.logging.impl.WeakHashtable.put(Object, Object) | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 240 |
Class org.apache.commons.logging.impl.WeakHashtable defines non-transient non-serializable instance field queue | BAD_PRACTICE | SE_BAD_FIELD | Not available |
org.apache.commons.logging.impl.WeakHashtable is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |