Skip to content

Conversation

@RyeMutt
Copy link
Contributor

@RyeMutt RyeMutt commented Nov 27, 2025

Description

Fixes an invalid symbol in gstreamer 1.0 plugin dlopen load crashing the plugin spuriously. I compared against the actual gstreamer header to confirm the api.

Related Issues

  • Please link to a relevant GitHub issue for additional context.
    • Bug Fix: Link to an issue that includes reproduction steps and testing guidance.
    • Feature/Enhancement: Link to an issue with a write-up, rationale, and requirements.

Issue Link:


Checklist

Please ensure the following before requesting review:

  • I have provided a clear title and detailed description for this pull request.
  • If useful, I have included media such as screenshots and video to show off my changes.
  • The PR is linked to a relevant issue with sufficient context.
  • I have tested the changes locally and verified they work as intended.
  • All new and existing tests pass.
  • Code follows the project's style guidelines.
  • Documentation has been updated if needed.
  • Any dependent changes have been merged and published in downstream modules
  • I have reviewed the contributing guidelines.

Additional Notes

Signed-off-by: Rye <rye@alchemyviewer.org>
@github-actions github-actions bot added the c/cpp label Nov 27, 2025
@akleshchev akleshchev requested a review from Geenz December 1, 2025 22:10
@Geenz
Copy link
Collaborator

Geenz commented Dec 1, 2025

@RyeMutt How would one test this?

@RyeMutt
Copy link
Contributor Author

RyeMutt commented Dec 1, 2025

I was seeing an occasional crash on plugin load under linux and tracked it down to trying to bind an invalid symbol. Verified correct symbol in gstreamer headers https://github.com/GStreamer/gstreamer/blob/1.26/subprojects/gstreamer/gst/gstutils.h#L1065

@Geenz Geenz merged commit b49400e into secondlife:develop-linux Dec 2, 2025
13 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants