[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [bluetooth-dev] Bug in Sdp.c?

At 12:35 00/06/30 +0200, Mats Frid$BqO(B wrote:
>We appreciate all feedback on the code, especially on the sdp which today is 
>more of a static function with a lot of hardcoded values.

>And finally, we are interested in all help with improving the sdp.

It seems to me that there are many thing to do:-)

Seeing the code, I have some opinions:

1.
SDP request PDU and respond PDU is one pair and it should be like:

typedef struct SdpPduPair {
	struct SdpPduPair  *pNextPduPair;  // points to next pair of cont. state portion
	SdpPdu_t           *pSdpReqPdu;
	SdpPdu_t           *pSdpRspPdu;
}SdpPduPair_t;

2.
SDP client and SDP server should be divided to avoid confusion.

Regards.