[PATCH 16/25] squash into 'network: convert local pointers to g_auto*'
Ján Tomko
jtomko at redhat.com
Thu Jun 25 23:34:10 UTC 2020
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.
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 */
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200626/2d0750ff/attachment-0001.sig>
More information about the libvir-list
mailing list