Fix bug in handling Session Keys
authorSimo Sorce <simo@redhat.com>
Thu, 3 Sep 2015 16:58:51 +0000 (12:58 -0400)
committerSimo Sorce <simo@redhat.com>
Thu, 3 Sep 2015 16:59:46 +0000 (12:59 -0400)
A check inversion in 86661d07812b010b8cf664c2dab596be15ff1e31 caused
the specified session key to be ignored and a crash when none was
specified.

Signed-off-by: Simo Sorce <simo@redhat.com>
src/mod_auth_gssapi.c

index 6057a44..a480997 100644 (file)
@@ -648,7 +648,7 @@ struct mag_req_cfg *mag_init_cfg(request_rec *req)
         req_cfg->desired_mechs = scfg->default_mechs;
     }
 
         req_cfg->desired_mechs = scfg->default_mechs;
     }
 
-    if (!req_cfg->cfg->mag_skey) {
+    if (req_cfg->cfg->mag_skey) {
         req_cfg->mag_skey = req_cfg->cfg->mag_skey;
     } else {
         /* Use server random key if not explicitly configured */
         req_cfg->mag_skey = req_cfg->cfg->mag_skey;
     } else {
         /* Use server random key if not explicitly configured */