LGTM is stupid and says we don't return a value in all code paths.

diff --git a/src/lib/eap_aka_sim/base.c b/src/lib/eap_aka_sim/base.c
index 03c4185398db44f6885cac215458e16aa2c75d6a..a0f5079efc3c6c809cef426536b39aa97560ab99 100644 (file)
--- a/src/lib/eap_aka_sim/base.c
+++ b/src/lib/eap_aka_sim/base.c
@@ -232,12 +232,14 @@ size_t fr_aka_sim_attr_len(fr_pair_t const *vp)
        case FR_TYPE_VARIABLE_SIZE:
                return vp->vp_length;
 
-       default:
-               return fr_aka_sim_attr_sizes[vp->vp_type][0];
-
        case FR_TYPE_STRUCTURAL:
                if (!fr_cond_assert(0)) return 0;
+
+       default:
+               break;
        }
+
+       return fr_aka_sim_attr_sizes[vp->vp_type][0];
 }
 
 /** Return the number of bytes before the octets value