Re: [PATCH 6/9] Add parser for transaction identifier objects

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1309 bytes --]

Hi Yang,

> ---
>  src/stkutil.c |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
> 
> diff --git a/src/stkutil.c b/src/stkutil.c
> index 3aae6bf..d573430 100644
> --- a/src/stkutil.c
> +++ b/src/stkutil.c
> @@ -748,6 +748,14 @@ static gboolean parse_dataobj_location_status(
>  					STK_DATA_OBJECT_TYPE_LOCATION_STATUS);
>  }
> 
> +/* Defined in TS 102.223 Section 8.28 */
> +static gboolean parse_dataobj_transaction_id(
> +		struct comprehension_tlv_iter *iter, void *user)
> +{
> +	return parse_dataobj_common_byte_array(iter, user,
> +				STK_DATA_OBJECT_TYPE_TRANSACTION_ID);
> +}
> +

Again, the maximum size is 127, so lets avoid a g_malloc here.

>  /* Defined in TS 102.223 Section 8.31 */
>  static gboolean parse_dataobj_icon_id(struct comprehension_tlv_iter *iter,
>  					void *user)
> @@ -873,6 +881,8 @@ static dataobj_handler handler_for_type(enum
>  stk_data_object_type type) return parse_dataobj_cause;
>  	case STK_DATA_OBJECT_TYPE_LOCATION_STATUS:
>  		return parse_dataobj_location_status;
> +	case STK_DATA_OBJECT_TYPE_TRANSACTION_ID:
> +		return parse_dataobj_transaction_id;
>  	case STK_DATA_OBJECT_TYPE_ICON_ID:
>  		return parse_dataobj_icon_id;
>  	case STK_DATA_OBJECT_TYPE_IMMEDIATE_RESPONSE:
> 

Regards,
-Denis