GNOME Bugzilla – Bug 787985
webpenc: fix unmap video frame in failed case
Last modified: 2017-09-23 08:08:58 UTC
Created attachment 360192 [details] [review] patch file attached Hi , In gst_webp_enc_handle_frame, missed Unmap video frame in failed case code: if (WebPEncode (&enc->webp_config, &enc->webp_picture)) { WebPPictureFree (&enc->webp_picture); out_buffer = gst_buffer_new_allocate (NULL, enc->webp_writer.size, NULL); if (!out_buffer) { GST_ERROR_OBJECT (enc, "Failed to create output buffer"); return GST_FLOW_ERROR; } gst_buffer_fill (out_buffer, 0, enc->webp_writer.mem, enc->webp_writer.size); free (enc->webp_writer.mem); } else { GST_ERROR_OBJECT (enc, "Failed to encode WebPPicture"); return GST_FLOW_ERROR; sol: gst_video_frame_unmap (&vframe); added Patch attached. Please review and share feedback.
Patch attached. Please review and share feedback.
Thanks. commit d99cef0b409936219cc123d18e4a1da711c8362e (HEAD -> master) Author: Ponnam Srinivas <p.srinivas@samsung.com> Date: Thu Sep 21 18:13:19 2017 +0530 webpenc: Unmap video frame in failed case https://bugzilla.gnome.org/show_bug.cgi?id=787985