Browse Source

Subscriptions: Set monitoredItemType in one place only

Julius Pfrommer 6 years ago
parent
commit
3d601763b9
1 changed files with 2 additions and 1 deletions
  1. 2 1
      src/server/ua_services_subscription.c

+ 2 - 1
src/server/ua_services_subscription.c

@@ -225,6 +225,8 @@ setMonitoredItemSettings(UA_Server *server, UA_MonitoredItem *mon,
     if(samplingInterval != samplingInterval) /* Check for nan */
         mon->samplingInterval = server->config.samplingIntervalLimits.min;
 
+    UA_assert(mon->monitoredItemType != 0);
+
     /* QueueSize */
     UA_BOUNDEDVALUE_SETWBOUNDS(server->config.queueSizeLimits,
                                params->queueSize, mon->maxQueueSize);
@@ -320,7 +322,6 @@ Operation_CreateMonitoredItem(UA_Server *server, UA_Session *session, struct cre
 
     /* Initialize the MonitoredItem */
     UA_MonitoredItem_init(newMon, cmc->sub);
-    newMon->monitoredItemType = UA_MONITOREDITEMTYPE_CHANGENOTIFY;
     newMon->attributeId = request->itemToMonitor.attributeId;
     newMon->timestampsToReturn = cmc->timestampsToReturn;
     UA_StatusCode retval = UA_STATUSCODE_GOOD;