Opened 6 years ago

Closed 6 years ago

#1726 closed defect (fixed)

Assertion in ICE connectivity check (thanks Amit Chowdhary for the report)

Reported by: nanang Owned by: bennylp
Priority: normal Milestone: release-2.2
Component: pjnath Version: trunk
Keywords: Cc:
Backport to 1.x milestone: Backported: no

Description

This seems to be related to changeset r4652, timer was always cancelled, but now it is not when mod_count==TRUE.

Reproducing step:

  1. run 2 pjsua with --use-ice param, no STUN/TURN
  2. make call from one to another, and just accept the call on the receiving side
  3. assertion happens on both sides once ICE connectivity check is started.

Assertion line:

Assertion failed: (tsx->retransmit_timer.id == 0 || !tsx->require_retransmit), function tsx_transmit_msg, file ../src/pjnath/stun_transaction.c, line 204.

Change History (1)

comment:1 Changed 6 years ago by nanang

  • Resolution set to fixed
  • Status changed from new to closed

In 4717:

Fix #1726: Added check in assertion line for transmitting STUN message without incrementing retransmission count.

Note: See TracTickets for help on using tickets.