-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Wed 13 Mar 2013 07:48:57 AM EDT, Sumit Bose wrote:
On Wed, Mar 13, 2013 at 12:32:05PM +0100, Pavel Březina wrote:
> From 78a1667c897d16ab7e08c49422160c2c5a64adf1 Mon Sep 17 00:00:00
> 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?=
> <pbrezina(a)redhat.com> Date: Wed, 13 Mar 2013 11:51:18 +0100
> Subject: [PATCH] krb5-utils-tests: remove invalid condition
>
> This condition is invalid because different_realm is not set,
> when EINVAL is returned. It can make the test fail sometimes.
> --- src/tests/krb5_utils-tests.c | 2 -- 1 file changed, 2
> deletions(-)
>
> diff --git a/src/tests/krb5_utils-tests.c
> b/src/tests/krb5_utils-tests.c index
> bc1890f0b5becf3f316eca31dfd20f30e25fa4f0..aa2f4ed4b8eb39c3aea8fdf53f27f12e096b9233
> 100644 --- a/src/tests/krb5_utils-tests.c +++
> b/src/tests/krb5_utils-tests.c @@ -700,8 +700,6 @@
> START_TEST(test_compare_principal_realm) ret =
> compare_principal_realm("userABC", "ABC", &different_realm);
> fail_unless(ret == EINVAL, "Missing '@' does not cause
> EINVAL.");
>
> - fail_unless(different_realm == false, "Same realm but " \ -
> "different_realm is not false."); ret =
> compare_principal_realm("user@ABC", "ABC", &different_realm);
> fail_unless(ret == EOK, "Failure with same realm");
> fail_unless(different_realm == false, "Same realm but " \ --
> 1.7.11.7
>
ACK
Thank you for catching this.
Pushed to master and sssd-1-9.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (GNU/Linux)
Comment: Using GnuPG with Thunderbird -
http://www.enigmail.net/
iEYEARECAAYFAlFAcnwACgkQeiVVYja6o6OmPACfXdW5t4FRfL23+dq1dRf5bWjj
mb8AnihsRHs8ijiUnu7U+Za2SCbhdqOg
=xdNH
-----END PGP SIGNATURE-----