[PATCH 16/25] squash into 'network: convert local pointers to g_auto*'

Laine Stump laine at redhat.com
Thu Jun 25 23:44:03 UTC 2020


On 6/25/20 7:34 PM, Ján Tomko wrote:
> On a Wednesday in 2020, Laine Stump wrote:
>> OOPS!!
>>
>> I meant to squash this into patch 10 before posting. If you want to 
>> just review it separately I can squash it in before push. Or if you 
>> want to be pedantic I can squash it in and resend :-)
>>
>
> To me, these seem like changes unrelated to patch 10 and deserve their
> own commit.


Well, they're all related to the cleanup that naturally follows from 
making a pointer g_autofree - 1) you need to initialize it when you 
define it (and so you might as well initialize it with the first real 
value it's going to get, as long as that value would end up *always* 
being assigned anyway, and 2) code reduction related to removing 
now-empty cleanup: label. But I see you just responded to patch 10 
saying you thought the patch was too long and should be split, so I 
suppose I could make this the 2nd of the 2 that you suggest.


>
> Jano
>
>> On 6/24/20 11:34 PM, Laine Stump wrote:
>>> Signed-off-by: Laine Stump <laine at redhat.com>
>>> ---
>>>  src/network/bridge_driver_linux.c | 9 ++-------
>>>  1 file changed, 2 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/src/network/bridge_driver_linux.c 
>>> b/src/network/bridge_driver_linux.c
>>> index 0d0ac730f2..7f765bcf99 100644
>>> --- a/src/network/bridge_driver_linux.c
>>> +++ b/src/network/bridge_driver_linux.c
>>> @@ -834,7 +834,7 @@ int networkAddFirewallRules(virNetworkDefPtr def)
>>>  {
>>>      size_t i;
>>>      virNetworkIPDefPtr ipdef;
>>> -    g_autoptr(virFirewall) fw = NULL;
>>> +    g_autoptr(virFirewall) fw = virFirewallNew();
>>>      if (virOnce(&createdOnce, networkSetupPrivateChains) < 0)
>>>          return -1;
>>> @@ -920,8 +920,6 @@ int networkAddFirewallRules(virNetworkDefPtr def)
>>>          }
>>>      }
>>> -    fw = virFirewallNew();
>>> -
>>>      virFirewallStartTransaction(fw, 0);
>>>      networkAddGeneralFirewallRules(fw, def);
>>> @@ -946,10 +944,7 @@ int networkAddFirewallRules(virNetworkDefPtr def)
>>>      virFirewallStartTransaction(fw, 
>>> VIR_FIREWALL_TRANSACTION_IGNORE_ERRORS);
>>>      networkAddChecksumFirewallRules(fw, def);
>>> -    if (virFirewallApply(fw) < 0)
>>> -        return -1;
>>> -
>>> -    return 0;
>>> +    return virFirewallApply(fw);
>>>  }
>>>  /* Remove all rules for all ip addresses (and general rules) on a 
>>> network */
>>
>>




More information about the libvir-list mailing list