From activemq-commits-return-3518-apmail-geronimo-activemq-commits-archive=geronimo.apache.org@geronimo.apache.org Tue Oct 03 14:26:43 2006 Return-Path: Delivered-To: apmail-geronimo-activemq-commits-archive@www.apache.org Received: (qmail 4507 invoked from network); 3 Oct 2006 14:26:43 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 3 Oct 2006 14:26:43 -0000 Received: (qmail 75903 invoked by uid 500); 3 Oct 2006 14:26:42 -0000 Delivered-To: apmail-geronimo-activemq-commits-archive@geronimo.apache.org Received: (qmail 75885 invoked by uid 500); 3 Oct 2006 14:26:42 -0000 Mailing-List: contact activemq-commits-help@geronimo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: activemq-dev@geronimo.apache.org Delivered-To: mailing list activemq-commits@geronimo.apache.org Received: (qmail 75876 invoked by uid 99); 3 Oct 2006 14:26:42 -0000 Received: from idunn.apache.osuosl.org (HELO idunn.apache.osuosl.org) (140.211.166.84) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 03 Oct 2006 07:26:42 -0700 X-ASF-Spam-Status: No, hits=-9.4 required=5.0 tests=ALL_TRUSTED,NO_REAL_NAME Received: from [140.211.166.113] ([140.211.166.113:63710] helo=eris.apache.org) by idunn.apache.osuosl.org (ecelerity 2.1.1.8 r(12930)) with ESMTP id F6/50-08153-12372254 for ; Tue, 03 Oct 2006 07:26:42 -0700 Received: by eris.apache.org (Postfix, from userid 65534) id 034971A981A; Tue, 3 Oct 2006 07:26:38 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r452507 - /incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java Date: Tue, 03 Oct 2006 14:26:38 -0000 To: activemq-commits@geronimo.apache.org From: jstrachan@apache.org X-Mailer: svnmailer-1.1.0 Message-Id: <20061003142639.034971A981A@eris.apache.org> X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Author: jstrachan Date: Tue Oct 3 07:26:38 2006 New Revision: 452507 URL: http://svn.apache.org/viewvc?view=rev&rev=452507 Log: added an explicit check that the destination is not null to prevent cryptic exception messages from the broker Modified: incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java Modified: incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java URL: http://svn.apache.org/viewvc/incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java?view=diff&rev=452507&r1=452506&r2=452507 ============================================================================== --- incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java (original) +++ incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java Tue Oct 3 07:26:38 2006 @@ -458,6 +458,9 @@ } else { throw new UnsupportedOperationException("This producer can only send messages to: " + this.info.getDestination().getPhysicalName()); } + if (dest == null) { + throw new JMSException("No destination specified"); + } this.session.send(this, dest, message, deliveryMode, priority, timeToLive); stats.onMessage();