On 11/3/25 3:14 PM, Hannes Laimer wrote:
> On 10/30/25 16:50, Stefan Hanreich wrote:
>> Used in the SDN zone bridges status endpoint for querying the members
>> of a specific bridge.
>>
>> Signed-off-by: Stefan Hanreich <[email protected]>
>> ---
>>   src/PVE/IPRoute2.pm | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/src/PVE/IPRoute2.pm b/src/PVE/IPRoute2.pm
>> index 70007f5..4e31376 100644
>> --- a/src/PVE/IPRoute2.pm
>> +++ b/src/PVE/IPRoute2.pm
>> @@ -32,6 +32,11 @@ sub ip_link_is_physical($ip_link) {
>>           && (!defined($ip_link->{linkinfo}) || !defined($ip_link-
>> >{linkinfo}->{info_kind}));
>>   }
>>   +sub ip_link_is_bridge_member($ip_link) {
>> +    return defined($ip_link->{linkinfo}->{info_slave_kind})
>> +        && $ip_link->{linkinfo}->{info_slave_kind} eq "bridge";
>> +}
>> +
> 
> we should probably check if `$ip_link->{linkinfo}` is undef here

yes, definitely makes sense - will do!


_______________________________________________
pve-devel mailing list
[email protected]
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to