From 5588433ad41e6e41c34eae3f0cba11c97d71180a Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Fri, 28 May 2010 10:57:37 -0500 Subject: [PATCH] stkutil: Fix style in build_dataobj_location_info --- src/stkutil.c | 39 +++++++++++++++++++++++++++------------ 1 file changed, 27 insertions(+), 12 deletions(-) diff --git a/src/stkutil.c b/src/stkutil.c index d93f6e84..d4aa19a0 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -3413,18 +3413,33 @@ static gboolean build_dataobj_location_info(struct stk_tlv_builder *tlv, sim_encode_mcc_mnc(mccmnc, li->mcc, li->mnc); - return stk_tlv_builder_open_container(tlv, cr, tag, FALSE) && - stk_tlv_builder_append_bytes(tlv, mccmnc, 3) && - stk_tlv_builder_append_short(tlv, li->lac_tac) && - (li->has_ci == FALSE || - stk_tlv_builder_append_short(tlv, li->ci)) && - (li->has_ext_ci == FALSE || - stk_tlv_builder_append_short(tlv, li->ext_ci)) && - (li->has_eutran_ci == FALSE || - (stk_tlv_builder_append_short(tlv, li->eutran_ci >> 12) && - stk_tlv_builder_append_short(tlv, - (li->eutran_ci << 4) | 0xf))) && - stk_tlv_builder_close_container(tlv); + if (stk_tlv_builder_open_container(tlv, cr, tag, FALSE) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_bytes(tlv, mccmnc, 3) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_short(tlv, li->lac_tac) == FALSE) + return FALSE; + + if (li->has_ci && stk_tlv_builder_append_short(tlv, li->ci) == FALSE) + return FALSE; + + if (li->has_ext_ci && + stk_tlv_builder_append_short(tlv, li->ext_ci) == FALSE) + return FALSE; + + if (li->has_eutran_ci) { + if (stk_tlv_builder_append_short(tlv, + li->eutran_ci >> 12) == FALSE) + return FALSE; + + if (stk_tlv_builder_append_short(tlv, + (li->eutran_ci << 4) | 0xf) == FALSE) + return FALSE; + } + + return stk_tlv_builder_close_container(tlv); } /* Described in TS 102.223 Section 8.20