Created attachment 917736 [details]
use calloc() for memory allocation
Description of problem:
Running omxregister-bellagio under valgrind produces a few warnings - use of uninitialised variables and memory leaks.
Ideally this should be fixed upstream, yet no-one has responded to my emails in almost a month :'(
Version-Release number of selected component (if applicable):
All (current latest version is 0.9.3-9 which exhibits the problems)
Steps to Reproduce:
1. Install libomxil-bellagio
2. Run valgrind omxregister-bellagio
Created attachment 917747 [details]
Plug memory leak of allNames
Humble monthly ping.
Can we please get these(or similar) patches included in the package ?
Another fortnight, another ping :)
Can anyone spare 5 minutes and pull these patches in, please ?
Two months have passed and no reply :'(
Would be great to have the issues addressed, even if one chooses different approach.
It's not a matter to "pull patches" , it's a matter to have the functionality tested. I wasn't able to test on any relevant hardware until recently. Now I need to find time to do the testing. (which might involve mesa patches since the omx backend was updated with mesa 10.3).
I also find memleak patch with omxregister tool as an extremely low priority at the point that I'm even reluctant to apply against any upstream resolution and QA on this isssue.
btw, which hardware/driver version was tested ?
Perhaps I should have been more explicit
- The issue is not hardware/driver related.
- One does _not_ need mesa, but only the libomxil-bellagio package.
Imho it's a matter of
- Install the libomxil-bellagio package.
- Run valgrind omxregister-bellagio
- Observe the issues pointed out
- Check if the patches correctly address the issues & QA
Bit curious - if you feel reluctant on plugging the memleak why did you request that I open a bugreport here in the first place :'(
Either way glad to see some progress on this.
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle.
Changing version to '22'.
More information and reason for this action is here:
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.
If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
Thank you for reporting this bug and we are sorry it could not be fixed.
Seems like this bug was closed without the proposed fix merged.
What's annoy me the most (as reported in the first comment) is that the upstream is dead for this project. And a big project like mesa using it does't seems like a good thing.
Better would be to have freedesktop or mesa to take-over maintenance of the libomxil-bellagio project).