sdk-5.2: wiced_tls_init_identity assertion bug

Tip / Sign in to post questions, reply, level up, and achieve exciting badges. Know more

cross mob
AxLi_1746341
Level 7
Level 7
10 comments on KBA 5 comments on KBA First comment on KBA

Test peap with join_ext command in console debug build test code will always hit assertion.

In peap case, join_ent() calls

wiced_tls_init_identity( &identity, NULL, 0, NULL, 0 );

So it always hit below assertion:

wiced_assert( "Bad args", (identity != NULL) && (private_key != NULL) && (certificate_data != NULL) );

And looking at the implementation in wiced_tls_init_identity:

Below checking looks wrong:

if ( ( certificate_data != NULL ) || ( certificate_length != 0 ) )

if ( ( private_key != NULL ) || ( key_length != 0 ) )

e.g. Pass NULL to private_key with non-zero key_length will hit null pointer dereference.

0 Likes
1 Solution

I believe this issue has been resolved so I will close this thread.

View solution in original post

0 Likes
3 Replies
GauravS_31
Moderator
Moderator
Moderator
10 questions asked 250 solutions authored 250 sign-ins

I have raised a ticket on this issue. You can comment out the wiced_assert( "Bad args", (identity != NULL) && (private_key != NULL) && (certificate_data != NULL) ); for now.

0 Likes

I have no problem to fix it by myself.

The reason I post it here is to make your sdk better.

I hope after you raised a ticket on the issue, this will be fixed in *next release*.

From the experience I found raise a ticket and get fixed has a very long gap.

For some issues, I even remind multiple times but still does not  get fix.

e.g.

https://community.cypress.com/thread/8502

https://community.cypress.com/thread/7781

https://community.cypress.com/thread/8102

Would you consider to improve your internal process?

0 Likes

I believe this issue has been resolved so I will close this thread.

0 Likes