Thanks. I’ve made this fix now.
FYI, the actual patch is simpler than the one that you proposed. It’s just:
diff -c RawVideoRTPSource.cpp~ RawVideoRTPSource.cpp
*** RawVideoRTPSource.cpp~ 2018-08-28 13:28:28.345447000 -0700
--- RawVideoRTPSource.cpp 2018-09-10 04:04:11.162413000 -0700
***************
*** 132,138 ****
// Make sure that we have enough bytes for all of the line lengths promised:
if (totalLength > packetSize) {
fNumLines = 0;
! delete[] fLineHeaders;
return False;
}
--- 132,138 ----
// Make sure that we have enough bytes for all of the line lengths promised:
if (totalLength > packetSize) {
fNumLines = 0;
! delete[] fLineHeaders; fLineHeaders = NULL;
return False;
}
because you don’t need to set
fLineHeaders = NULL;
in the destructor, because “fLineHeaders” is never used again.
Ross Finlayson
Live Networks, Inc.
http://www.live555.com/
_______________________________________________
live-devel mailing list
live-devel@lists.live555.com
http://lists.live555.com/mailman/listinfo/live-devel
