rtspsrc: Avoid stack overflow recursing waiting for response (!561) · Merge requests · GStreamer / gst-plugins-good · GitLab

Snippets Groups Projects

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

The migration is almost done, at least the rest should happen in the background. There are still a few technical difference between the old cluster and the new ones, and they are summarized in this issue. Please pay attention to the TL:DR at the end of the comment.

Merged Thibault Saunier requested to merge thiblahute/gst-plugins-good:rtspsrc_avoid_recursing_to_much into master 4 years ago

Instead of recursing, simply implement a loop with gotos, the same way it was done before 81217528

Fixes #710 (closed)

Activity

Thibault Saunier added Maybe backport label 4 years ago

added Maybe backport label
Sebastian Dröge added 1.16 Needs backport labels and removed Maybe backport label 4 years ago

added 1.16 Needs backport labels and removed Maybe backport label
Sebastian Dröge assigned to @gstreamer-merge-bot 4 years ago

assigned to @gstreamer-merge-bot
Sebastian Dröge @slomo · 4 years ago

Owner

It's even tail recursive, too bad that TCE is not guaranteed by C...
GStreamer Marge Bot merged 4 years ago

merged
GStreamer Release Monkey changed milestone to %1.17.1 4 years ago

changed milestone to %1.17.1
Tim-Philipp Müller added Backported into 1.16 label and removed Needs backport label 4 years ago

added Backported into 1.16 label and removed Needs backport label
Tim-Philipp Müller @tpm · 4 years ago

Owner

Merge Request for backport into 1.16 branch: !751 (merged) (target milestone: 1.16.3)

Needs manual intervention!
Tim-Philipp Müller mentioned in merge request !751 (merged) 4 years ago

mentioned in merge request !751 (merged)

Please register or sign in to reply