[Bug 731943] pnmenc: PNM Encoder needs to use gstVideoEncoder as base class

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=731943
  GStreamer | gst-plugins-bad | unspecified

Thiago Sousa Santos <thiagossantos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |thiagossantos at gmail.com
     Ever Confirmed|0                           |1

--- Comment #3 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-07-21 21:00:35 UTC ---
A few things to fix:

1) When submitting a new version of a patch, please resubmit the full patch
instead of an amendment patch, this makes easier for reviewers to try and push.

2) There seems to be some memory corruption in this ported version as the
command:
"gst-launch-1.0 videotestsrc num-buffers=5 ! pnmenc ! multifilesink
location=/tmp/test_%02d.pnm"

Aborts with some issue inside malloc. Using valgrind should easily point out
the failure.

Try to run valgrind with something like:

G_DEBUG=gc-friendly G_SLICE=always-malloc valgrind --trace-children=yes
gst-launch-1.0 videotestsrc num-buffers=5 ! pnmenc ! multifilesink
location=/tmp/test_%02d.pnm

and look for "Invalid write" entries.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.