Browse code

textops: RFC2543 is broken

commit cbcc27ef8b1674448bd8b1c45f03388e36f65f51 broke RFC2543 hold

there's no need to check for sendrecv_mode, is_on_hold is either set or not in sdp_stream and/or sdp_session

lazedo authored on 06/09/2021 23:35:52
Showing 1 changed files
... ...
@@ -4517,13 +4517,10 @@ static int ki_is_audio_on_hold(sip_msg_t *msg)
4517 4517
 				if(!sdp_stream) break;
4518 4518
 				if(sdp_stream->media.len==AUDIO_STR_LEN &&
4519 4519
 					strncmp(sdp_stream->media.s,AUDIO_STR,AUDIO_STR_LEN)==0 &&
4520
-					sdp_stream->sendrecv_mode.len &&
4521 4520
 					sdp_stream->is_on_hold)
4522 4521
 					return sdp_stream->is_on_hold;
4523 4522
 				if(sdp_stream->media.len==AUDIO_STR_LEN &&
4524 4523
 					strncmp(sdp_stream->media.s,AUDIO_STR,AUDIO_STR_LEN)==0 &&
4525
-					!sdp_stream->sendrecv_mode.len &&
4526
-					sdp_session->sendrecv_mode.len &&
4527 4524
 					sdp_session->is_on_hold)
4528 4525
 					return sdp_session->is_on_hold;
4529 4526
 				sdp_stream_num++;