NormalizedMessageImpl.getAttachment() returns DataHandler when it should return 
null
------------------------------------------------------------------------------------

                 Key: SM-1186
                 URL: https://issues.apache.org/activemq/browse/SM-1186
             Project: ServiceMix
          Issue Type: Bug
          Components: servicemix-core
    Affects Versions: 3.2.1
         Environment: any
            Reporter: Alex Vigdor


The current implementation of getAttachment() in NormalizedMessageImpl always 
returns a DataHandler if the message has any attachments, even if it does not 
have an attachment with the given ID:

    public DataHandler getAttachment(String id) {
        if (attachments != null) {
            return new DataHandler((DataSource) attachments.get(id));
        }
        return null;
    }

According to the JBI javadocs, getAttachment should return "DataHandler 
representing attachment content, or null if an attachment with the specified 
identifier is not found".

Thus, the implementation should add an additional check

if(attachments !=null && attachments.get(id) != null){
    return new DataHandler((DataSource) attachments.get(id));
}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to