[Asterisk-code-review] SDP: Ensure SDPs "merge" properly. (asterisk[master])

[Joshua Colp]

Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/5415 )

Change subject: SDP: Ensure SDPs "merge" properly.
......................................................................


Patch Set 2:

(2 comments)

https://gerrit.asterisk.org/#/c/5415/2/main/sdp_state.c
File main/sdp_state.c:

Line 578: 			joint_stream = ast_stream_alloc("dummy", new_stream_type);
> Yes, most likely. And that's probably not a good thing.
It's okay for it to be there, provided a saneish name is used and the proper state is set on the stream.


Line 806: 		if (state_stream->instance) {
> I counter that only an audio or video stream will have an RTP instance due 
It's a union though so another pointer may go into that place.


-- 
To view, visit https://gerrit.asterisk.org/5415
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5938c2be3c6f0a003aa88a39a59e0880f8b2df3d
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-HasComments: Yes