[jira] [Commented] (ARTEMIS-1128) QueueControlImpl.sendMessage() fixes

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARTEMIS-1128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15981290#comment-15981290 ] 

ASF GitHub Bot commented on ARTEMIS-1128:
-----------------------------------------

GitHub user gaohoward opened a pull request:

    https://github.com/apache/activemq-artemis/pull/1222

    ARTEMIS-1128 QueueControlImpl.sendMessage() fixes

    sendMessage() may throw ActiveMQException that causes CNFE
    at the management client. Also it should check if headers
    in the message is null (to prevent NPE).

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gaohoward/activemq-artemis master_fix_jon

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/1222.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1222
    
----

----


> QueueControlImpl.sendMessage() fixes
> ------------------------------------
>
>                 Key: ARTEMIS-1128
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1128
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 2.0.0
>            Reporter: Howard Gao
>            Assignee: Howard Gao
>             Fix For: 2.next
>
>
> sendMessage() may throw ActiveMQException that causes CNFE at the management client. Also it should check if headers in the message is null (to prevent NPE).



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)