Hi,
I found this potential crash when trying to find another issue in the failover code. To reproduce, just revert the changes to src/providers/fail_over.c and run make check, you should see either a crash or at least an error if you use valgrind.
On Thu, Nov 19, 2015 at 05:35:07PM +0100, Jakub Hrozek wrote:
--- a/src/tests/cmocka/test_fo_srv.c +++ b/src/tests/cmocka/test_fo_srv.c @@ -203,6 +203,7 @@ struct test_fo_ctx { int ttl;
struct fo_server *srv;
- int parallel_res;
};
This variable was unused so the attached patch version removes it.
On 11/23/2015 06:39 PM, Jakub Hrozek wrote:
On Thu, Nov 19, 2015 at 05:35:07PM +0100, Jakub Hrozek wrote:
--- a/src/tests/cmocka/test_fo_srv.c +++ b/src/tests/cmocka/test_fo_srv.c @@ -203,6 +203,7 @@ struct test_fo_ctx { int ttl;
struct fo_server *srv;
- int parallel_res; };
This variable was unused so the attached patch version removes it.
Ack.
On Thu, Dec 03, 2015 at 02:24:19PM +0100, Pavel Březina wrote:
On 11/23/2015 06:39 PM, Jakub Hrozek wrote:
On Thu, Nov 19, 2015 at 05:35:07PM +0100, Jakub Hrozek wrote:
--- a/src/tests/cmocka/test_fo_srv.c +++ b/src/tests/cmocka/test_fo_srv.c @@ -203,6 +203,7 @@ struct test_fo_ctx { int ttl;
struct fo_server *srv;
- int parallel_res;
};
This variable was unused so the attached patch version removes it.
Ack.
* master: a92f68763a57b211a1bf6b80b6dd80c4a1aa2738
On (07/12/15 16:15), Jakub Hrozek wrote:
On Thu, Dec 03, 2015 at 02:24:19PM +0100, Pavel Březina wrote:
On 11/23/2015 06:39 PM, Jakub Hrozek wrote:
On Thu, Nov 19, 2015 at 05:35:07PM +0100, Jakub Hrozek wrote:
--- a/src/tests/cmocka/test_fo_srv.c +++ b/src/tests/cmocka/test_fo_srv.c @@ -203,6 +203,7 @@ struct test_fo_ctx { int ttl;
struct fo_server *srv;
- int parallel_res;
};
This variable was unused so the attached patch version removes it.
Ack.
- master: a92f68763a57b211a1bf6b80b6dd80c4a1aa2738
sssd-1-13: * bbfc3a4294c6a70426171e080c27c15d9706bec7
LS
sssd-devel@lists.fedorahosted.org
-
Jakub Hrozek -
Lukas Slebodnik -
Pavel Březina