Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • gstreamer gstreamer
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 1.1k
    • Issues 1.1k
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 499
    • Merge requests 499
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GStreamerGStreamer
  • gstreamergstreamer
  • Merge requests
  • !1037

audioaggregator: Use an ad-hoc converter when changing srcpad caps

  • Review changes

  • Download
  • Patches
  • Plain diff
Open Jan Alexander Steffens requested to merge heftig/gstreamer:gst-plugins-base-audioaggregator-caps-change-crash into main Oct 04, 2021
  • Overview 2
  • Commits 2
  • Pipelines 2
  • Changes 1

Copied from https://gitlab.freedesktop.org//gstreamer/gst-plugins-base/-/merge_requests/1117

The sinkpad converters created by gst_audio_aggregator_convert_pad_update_converter cannot be used:

  • The converter-config could be invalid for this conversion. For example, applying the mix-matrix to this internal conversion makes no sense.

  • The function resets converter_config_changed, assuming that the the provided infos are for the sinkpad → srcpad conversion. This can leave the pad in a bad state, set up with the wrong resampler, causing a buffer overflow when it next attempts this conversion.

FIXME: This still feels really brittle and could cause glitches when the conversion doesn't match what the sinkpad converters would have created for the new caps. I think the audioaggregator should probably keep the pre-conversion sinkpad buffers around and re-convert to the new caps instead of attempting to convert between internal formats.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gst-plugins-base-audioaggregator-caps-change-crash