From ed201b3bca9b3c09895a0c25eedbbf8314eb5f57 Mon Sep 17 00:00:00 2001 From: "Alan T. DeKok" Date: Tue, 20 Sep 2016 07:44:41 -0400 Subject: [PATCH] use defines for shift, instead of hard-coded number --- src/modules/rlm_eap/types/rlm_eap_fast/rlm_eap_fast.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/rlm_eap/types/rlm_eap_fast/rlm_eap_fast.c b/src/modules/rlm_eap/types/rlm_eap_fast/rlm_eap_fast.c index be9c92f..9911ce4 100644 --- a/src/modules/rlm_eap/types/rlm_eap_fast/rlm_eap_fast.c +++ b/src/modules/rlm_eap/types/rlm_eap_fast/rlm_eap_fast.c @@ -305,7 +305,7 @@ error: for (VALUE_PAIR *vp = fr_cursor_init(&cursor, &fast_vps); vp; vp = fr_cursor_next(&cursor)) { char *value; - switch (vp->da->attr >> 24) { + switch ((vp->da->attr >> fr_attr_shift[3]) & fr_attr_mask[3]) { case PAC_INFO_PAC_TYPE: rad_assert(t->pac.type == 0); t->pac.type = vp->vp_integer; -- 2.1.4