[Bug]: Seemingly SessionDescription creation bug

[wei-zhang-simplisafe]

Please confirm you have already done the following

• I have searched the repository for related/existing bug reports
• I have all the details the issue requires

Please answer the following prompt

• This issue is replicable using the unmodified sample application

Describe the bug

This line looks wrong:
https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/blob/master/src/source/PeerConnection/SessionDescription.c?plain=1#L659
The SPRTINTF will be overwritten by the next SPRINTF:
https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/blob/master/src/source/PeerConnection/SessionDescription.c?plain=1#L662

nack feature is supported by webrtc SDK, so these lines should be in the SDP:
a=rtcp-fb:125 nack
a=rtcp-fb:125 nack pli

But because of the above issue, only "a=rtcp-fb:125 nack pli" is in the SDP.
Some peer doesn't know webrtc SDK supports NACK, so won't send NACK RTCP packets.

Expected Behavior

These lines should be in the SDP:
a=rtcp-fb:125 nack
a=rtcp-fb:125 nack pli

Current Behavior

Only a=rtcp-fb:125 nack pli is in the SDP

Reproduction Steps

Run sample code, and check the SDP text

WebRTC C SDK version being used

Most recent code base

If it was working in a previous version, which one?

No

Compiler and Version used

Any compiler

Operating System and version

any OS

Platform being used

any platform

[disa6302]

It does look like a bug. Thanks @wei-zhang-simplisafe ! Will get this fixed.

[hassanctech]

fix has been merged!