bkuner / opcUaUnifiedAutomation

EPICS opcUa device support with Unified Automation C++ based client sdk.
Other
0 stars 4 forks source link

Link string parser does not allow ':' in identifier #7

Closed ralphlange closed 7 years ago

ralphlange commented 7 years ago

The link string parsing in DevUaClient::getNodeFromBrowsePath() does not allow using a colon ':' character in an identifier. Instead, the ':' is interpreted as the separator between namespace index and browsepath.

A link that defines

@0,System1:OPC-UADP1

yields

[...]
DevUaClient::getNodeFromBrowsePath
    parsed NS:0 PATH: OPC-UADP1

The parser should be strictly checking for the first character after a leading integer and report an error if it is not a ',' or a ':'. (In case of an 'automatic' mode that tries both options, the tag characters must be something that is not allowed in browse path or identifier.)