<p dir="ltr"><br>
On Apr 3, 2015 2:27 PM, "Dominik Taborsky" <<a href="mailto:dominik.taborsky@nic.cz">dominik.taborsky@nic.cz</a>> wrote:<br>
><br>
> Any thoughts on this?<br>
><br>
> If anything, could anyone point me the right direction so I can fix it myself?<br>
><br>
><br>
><br>
> On 03/25/2015 11:28 AM, Dominik Taborsky wrote:<br>
>><br>
>> Hello,<br>
>><br>
>> a colleague of mine discovered a problem when using UCI: when there's a clash of identifiers, they mix together regardless of their type and the last type is set as the type of the identifier. Let me show an example:<br>
>><br>
>> root@turris:/etc/config# cat conflicting_config<br>
>> config typeA 'sec1'<br>
>>         list device 'dev1'<br>
>>         list ignore 'dev2'<br>
>><br>
>> config typeA 'sec2'<br>
>>         list device 'dev2'<br>
>>         list ignore 'dev3'<br>
>><br>
>> config typeB 'sec1'<br>
>>         list activate 'dev3'<br>
>>         list activate 'dev4'<br>
>>         option socket 'false'<br>
>></p>
<p dir="ltr">Section names are supposed to be unique within a uci package unit since we allow actions such as "uci set pkgname.secname.optname=xxx".</p>
<p dir="ltr">>> root@turris:/etc/config# uci show conflicting_config<br>
>> conflicting_config.sec1=typeB<br>
>> conflicting_config.sec1.device=dev1<br>
>> conflicting_config.sec1.ignore=dev2<br>
>> conflicting_config.sec1.activate=dev3 dev4<br>
>> conflicting_config.sec1.socket=false<br>
>> conflicting_config.sec2=typeA<br>
>> conflicting_config.sec2.device=dev2<br>
>> conflicting_config.sec2.ignore=dev3<br>
>><br>
>><br>
>> root@turris:/etc/config# uci show conflicting_config.@typeA[0]<br>
>> conflicting_config.sec2=typeA<br>
>> conflicting_config.sec2.device=dev2<br>
>> conflicting_config.sec2.ignore=dev3<br>
>><br>
>><br>
>> As you can see, UCI shows only one section named 'sec1' of typeB, while there are two sections of that name of different types. It mixes the options and lists together and does not report any problem with this. I would expect either to report a problem or consider these as different objects with the same name. A workaround for this is to use anonymous sections with an option with the name as the value, but I still think there at least should be a warning. I don't know how to report such a problem from within the UCI library though.<br>
>></p>
<p dir="ltr">How about warning on command line in normal mode and error when parsing in strict mode?</p>
<p dir="ltr">cheers,<br>
                yousong</p>