Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • tracker-miners tracker-miners
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 96
    • Issues 96
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 20
    • Merge requests 20
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
    • Model experiments
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • tracker-minerstracker-miners
  • Issues
  • #87

Fix or remove gstreamer generic image extraction path

We have a '90-gstreamer-image-generic.rule' extract rule which can fail in damaging ways.

The underlying cause is https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/691 -- a GstDiscoverer misfeature where it decides that unknown files must be MP3s and feeds large amounts of data into an MPEG decoder, which causes very high CPU and IO usage and sometimes causes Linux to hang altogether.

The two recent issues are:

  • tracker#158 (closed)
  • !120 (merged)

This extractor is of dubious utility anyway. In the best case it'll only manage to return the width and height of the image, which is not very useful information.

I propose for 3.0 we remove this extractor as I don't see a use case.

Another option, which would go a long way to reduce the risk of causing system lockups, would be reducing the timeout we pass to gst_discoverer_new() from 5 seconds to 1 second.

Assignee
Assign to
Time tracking