Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1eae32e2 authored by Kim Schulz's avatar Kim Schulz Committed by Zhihai Xu
Browse files

fixed problem with parsing GetAllCapabilitiesRsp 

The check for GetAllCapabilitiesRsp was not correctly parsed and hence the qualification tests TP/SIG/SMG/BI-35-C and TP/SIG/SMG/BI-36-C could not be passed.
This fixes that problem and makes it comply with the spec.

Bug: 10906942
Change-Id: I8b46c1974b4cad13d5a6641e02bf8a5f2ebc8ebd
parent 285ec59a

system/stack/avdt/avdt_msg.c

+6 −2
Original line number Diff line number Diff line
@@ -688,8 +688,12 @@ static UINT8 avdt_msg_prs_cfg(tAVDT_CFG *p_cfg, UINT8 *p, UINT16 len, UINT8* p_e 
                /* verify length */

                AVDT_TRACE_WARNING2("psc_mask=0x%x elem_len=%d", p_cfg->psc_mask, elem_len);

                if( ((0 == (p_cfg->psc_mask & (AVDT_PSC_RECOV|AVDT_PSC_REPORT))) && (elem_len != 3))

                    || ((p_cfg->psc_mask & AVDT_PSC_RECOV) && (elem_len != 7))

                    || ((p_cfg->psc_mask & AVDT_PSC_REPORT) && (elem_len != 5)) )

                    || (((p_cfg->psc_mask & AVDT_PSC_REPORT) && !(p_cfg->psc_mask & AVDT_PSC_RECOV))

                    && (elem_len != 5))

                    || ((!(p_cfg->psc_mask & AVDT_PSC_REPORT) && (p_cfg->psc_mask & AVDT_PSC_RECOV))

                    && (elem_len != 5))

                    || (((p_cfg->psc_mask & AVDT_PSC_REPORT) && (p_cfg->psc_mask & AVDT_PSC_RECOV))

                    && (elem_len != 7)) )

                {

                    err = AVDT_ERR_MUX_FMT;

                    break;