mediasoup icon indicating copy to clipboard operation
mediasoup copied to clipboard

[SCTP] Socket (part 2)

Open ibc opened this issue 7 months ago • 2 comments

Details

  • Progress in SCTP Socket class.
  • Many improvements and additions in SCTP Chunk/Parameter subclasses.
  • Add handles/BackoffTimerHandle which internally manages a TimerHandler.

ibc avatar May 07 '25 09:05 ibc

@jmillan please take a look specially to the new handles/BackoffTimerHandle class.

ibc avatar May 09 '25 22:05 ibc

Ok a possible bug here: https://github.com/versatica/mediasoup/actions/runs/17612408708/job/50037215122?pr=1533

../../../src/RTC/SCTP/packet/TLV.cpp:115:50: runtime error: null pointer passed as argument 2, which is declared to never be null
/usr/include/string.h:48:14: note: nonnull attribute specified here
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../../../src/RTC/SCTP/packet/TLV.cpp:115:50 

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mediasoup-worker-test-asan-undefined is a Catch2 v3.8.1 host application.
Run with -? for options

I can reproduce a similar issue in Docker by running:

MESON_ARGS="-Dms_log_trace=true -Dms_log_file_line=true -Dms_disable_liburing=true -Dms_sctp_stack=true" make test-asan-undefined
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../../../include/RTC/SCTP/packet/Parameter.hpp:145:65 
../../../src/RTC/SCTP/packet/TLV.cpp:115:50: runtime error: null pointer passed as argument 2, which is declared to never be null
/usr/include/string.h:48:14: note: nonnull attribute specified here

ibc avatar Sep 11 '25 08:09 ibc