Am 23.10.18 um 20:10 schrieb Michael Biebl:
> Hi Bernhard
> 
> Am 23.10.18 um 18:45 schrieb Bernhard Übelacker:
>> Hello all,
>>
>> I tried to reproduce this issue and think I found the problem.
>>
>> In commit [1] a typo creeped in and "block->name" got replaced by 
>> "block_name".
>> Variable block_name gets not initialized and therefore g_str_has_prefix 
>> crashes.
>> Might be on other architectures just valid or zero by luck.
>>
>>                      /* Bridge configuration */
>> -                    if(!strncmp ("br", block->name, 2)) {
>> +                    if (g_str_has_prefix (block_name, "br")) {
>>                              /* Try to find bridge ports */
>>
> 
> Well, spotted. That looks indeed like the culprit.

This desperately cries for some autopkgtest tests which tests various
ifupdown configurations
- managed=true/false
- valid iface config with/without a corresponding auto/allow-hotplug
- invalid iface config with/without a corresponding auto/allow-hotplug
- ...

Lot's of combinations to test...

I'd be more then grateful if someone want's to give NM a bit of
autopkgtest love.

Michael
-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to