[libvirt] [PATCH v3 0/2] lxc: Fix a bug related to IPv{4, 6} gateway persistent setting.

Julio Faracco posted 2 patches 5 years, 3 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20181130124337.25391-1-jcfaracco@gmail.com
src/lxc/lxc_native.c                          | 27 ++++++-----
.../lxcconf2xml-miscnetwork-v3.config         | 23 ++++++++++
.../lxcconf2xml-miscnetwork.config            | 23 ++++++++++
.../lxcconf2xml-miscnetwork.xml               | 45 +++++++++++++++++++
tests/lxcconf2xmltest.c                       |  2 +
5 files changed, 106 insertions(+), 14 deletions(-)
create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork-v3.config
create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.config
create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.xml
[libvirt] [PATCH v3 0/2] lxc: Fix a bug related to IPv{4, 6} gateway persistent setting.
Posted by Julio Faracco 5 years, 3 months ago
This serie fixes a bug related to IPv{4,6} gateway settings when it is
defined and used with multiple network definitions. Basically, this data
is being carried on to the next network settings because the pointer is
not being cleaned up/initialized properly. The idea behind the fix was
create a new way to initialize the data without knowing the structure
attributes. The old way has a high probability to cause new bugs.

This serie add a new test case to cover this scenario too. It will be so
important to network index implemented on LXC 3.X.

v1-v2: Fixing tabs inside the code.

Julio Faracco (2):
  lxc: Initializing IPv6 and IPv4 gateway to overwrite old settings.
  tests: Adding test case to include multiple network definitions.

 src/lxc/lxc_native.c                          | 27 ++++++-----
 .../lxcconf2xml-miscnetwork-v3.config         | 23 ++++++++++
 .../lxcconf2xml-miscnetwork.config            | 23 ++++++++++
 .../lxcconf2xml-miscnetwork.xml               | 45 +++++++++++++++++++
 tests/lxcconf2xmltest.c                       |  2 +
 5 files changed, 106 insertions(+), 14 deletions(-)
 create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork-v3.config
 create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.config
 create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.xml

-- 
2.19.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v3 0/2] lxc: Fix a bug related to IPv{4, 6} gateway persistent setting.
Posted by John Ferlan 5 years, 3 months ago

On 11/30/18 7:43 AM, Julio Faracco wrote:
> This serie fixes a bug related to IPv{4,6} gateway settings when it is
> defined and used with multiple network definitions. Basically, this data
> is being carried on to the next network settings because the pointer is
> not being cleaned up/initialized properly. The idea behind the fix was
> create a new way to initialize the data without knowing the structure
> attributes. The old way has a high probability to cause new bugs.
> 
> This serie add a new test case to cover this scenario too. It will be so
> important to network index implemented on LXC 3.X.
> 
> v1-v2: Fixing tabs inside the code.
> 
> Julio Faracco (2):
>   lxc: Initializing IPv6 and IPv4 gateway to overwrite old settings.
>   tests: Adding test case to include multiple network definitions.
> 
>  src/lxc/lxc_native.c                          | 27 ++++++-----
>  .../lxcconf2xml-miscnetwork-v3.config         | 23 ++++++++++
>  .../lxcconf2xml-miscnetwork.config            | 23 ++++++++++
>  .../lxcconf2xml-miscnetwork.xml               | 45 +++++++++++++++++++
>  tests/lxcconf2xmltest.c                       |  2 +
>  5 files changed, 106 insertions(+), 14 deletions(-)
>  create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork-v3.config
>  create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.config
>  create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-miscnetwork.xml
> 

Reviewed-by: John Ferlan <jferlan@redhat.com>

(and pushed),

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list