This fixes regression caused by the 1d39dbaf637db03f6e597ed56b96aa065710b4a1 fdlist[i] erroneously was replaced by fdlist[1] which caused lxcOpenNamespace to return a list with identical elements. --- libvirt-lxc-override.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libvirt-lxc-override.c b/libvirt-lxc-override.c index 60c2e48..d7af154 100644 --- a/libvirt-lxc-override.c +++ b/libvirt-lxc-override.c @@ -83,7 +83,7 @@ libvirt_lxc_virDomainLxcOpenNamespace(PyObject *self ATTRIBUTE_UNUSED, goto error; for (i = 0; i < c_retval; i++) - VIR_PY_LIST_APPEND_GOTO(py_retval, libvirt_intWrap(fdlist[1]), error); + VIR_PY_LIST_APPEND_GOTO(py_retval, libvirt_intWrap(fdlist[i]), error); cleanup: VIR_FREE(fdlist);

Thanks for catching this! However to be able to catch this, per our rules: https://libvirt.org/governance.html#contributors we need you to provide a sign-off to certify you complied with the Developer Certificate of Origin: https://developercertificate.org/ Jano

>> --- >>  libvirt-lxc-override.c | 2 +- >>  1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libvirt-lxc-override.c b/libvirt-lxc-override.c >> index 60c2e48..d7af154 100644 >> --- a/libvirt-lxc-override.c >> +++ b/libvirt-lxc-override.c >> @@ -83,7 +83,7 @@ libvirt_lxc_virDomainLxcOpenNamespace(PyObject >> *self ATTRIBUTE_UNUSED, >>          goto error; >>      for (i = 0; i < c_retval; i++) >> -        VIR_PY_LIST_APPEND_GOTO(py_retval, >> libvirt_intWrap(fdlist[1]), error); >> +        VIR_PY_LIST_APPEND_GOTO(py_retval, >> libvirt_intWrap(fdlist[i]), error); >>   cleanup: >>      VIR_FREE(fdlist); > > > -- > libvir-list mailing list > libvir-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/libvir-list -- libvir-list mailing list libvir-list(a)redhat.com https://www.redhat.com/mailman/listinfo/libvir-list