Unlock only on return

Signed-off-by: Derek Collison <derek@nats.io>

server/consumer.go

@@ -606,11 +606,11 @@ func (mset *stream) addConsumerWithAssignment(config *ConsumerConfig, oname stri
 			// Check in place here for interest. Will setup properly in setLeader.
 			r := o.acc.sl.Match(o.cfg.DeliverSubject)
 			if !o.hasDeliveryInterest(len(r.psubs)+len(r.qsubs) > 0) {
-				mset.mu.Unlock()
 				// Directs can let the interest come to us eventually, but setup delete timer.
 				if config.Direct {
 					o.updateDeliveryInterest(false)
 				} else {
+					mset.mu.Unlock()
 					o.deleteWithoutAdvisory()
 					return nil, errNoInterest
 				}