Mercurial > mplayer.hg
annotate libmpcodecs/vf_screenshot.c @ 36757:18518cb8a99e
vf_screenshot: Fix memory leak on resolution/aspect changes.
author | reimar |
---|---|
date | Sun, 16 Feb 2014 14:47:36 +0000 |
parents | 6e78b4f9569f |
children | 0fa1993a31c3 |
rev | line source |
---|---|
30421
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
1 /* |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
2 * This file is part of MPlayer. |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
3 * |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
4 * MPlayer is free software; you can redistribute it and/or modify |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
5 * it under the terms of the GNU General Public License as published by |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
6 * the Free Software Foundation; either version 2 of the License, or |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
7 * (at your option) any later version. |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
8 * |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
9 * MPlayer is distributed in the hope that it will be useful, |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
12 * GNU General Public License for more details. |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
13 * |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
14 * You should have received a copy of the GNU General Public License along |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
15 * with MPlayer; if not, write to the Free Software Foundation, Inc., |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
16 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
17 */ |
bbb6ebec87a0
Add missing license headers to all files in the libmpcodecs directory.
diego
parents:
30150
diff
changeset
|
18 |
17012 | 19 #include "config.h" |
16446 | 20 |
21 #include <stdio.h> | |
22 #include <stdlib.h> | |
23 #include <string.h> | |
24 #include <inttypes.h> | |
25 | |
26 #include <sys/types.h> | |
27 #include <sys/stat.h> | |
28 #include <unistd.h> | |
29 | |
17012 | 30 #include "mp_msg.h" |
16446 | 31 |
32 #include "img_format.h" | |
33 #include "mp_image.h" | |
34 #include "vf.h" | |
35 #include "vf_scale.h" | |
36 | |
35712
d206960484fe
Add a number of missing libavutil header #includes.
diego
parents:
35114
diff
changeset
|
37 #include "libavutil/mem.h" |
18861 | 38 #include "libswscale/swscale.h" |
24731 | 39 #include "libavcodec/avcodec.h" |
40 | |
16446 | 41 struct vf_priv_s { |
42 int frameno; | |
36752
af146b0d33f0
Allow specifying custom path and name for screenshot files.
reimar
parents:
36751
diff
changeset
|
43 char fname[PATH_MAX]; |
af146b0d33f0
Allow specifying custom path and name for screenshot files.
reimar
parents:
36751
diff
changeset
|
44 char *prefix; |
19833 | 45 /// shot stores current screenshot mode: |
46 /// 0: don't take screenshots | |
47 /// 1: take single screenshot, reset to 0 afterwards | |
48 /// 2: take screenshots of each frame | |
16450 | 49 int shot, store_slices; |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
50 int dw, dh; |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
51 AVFrame *pic; |
16446 | 52 struct SwsContext *ctx; |
24731 | 53 AVCodecContext *avctx; |
54 uint8_t *outbuffer; | |
55 int outbuffer_size; | |
16446 | 56 }; |
57 | |
58 //===========================================================================// | |
59 | |
36309 | 60 static void draw_slice(struct vf_instance *vf, unsigned char** src, |
61 int* stride, int w,int h, int x, int y) | |
62 { | |
63 if (vf->priv->store_slices) { | |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
64 sws_scale(vf->priv->ctx, src, stride, y, h, vf->priv->pic->data, vf->priv->pic->linesize); |
36309 | 65 } |
66 vf_next_draw_slice(vf,src,stride,w,h,x,y); | |
67 } | |
68 | |
30642
a972c1a4a012
cosmetics: Rename struct vf_instance_s --> vf_instance.
diego
parents:
30638
diff
changeset
|
69 static int config(struct vf_instance *vf, |
27947 | 70 int width, int height, int d_width, int d_height, |
71 unsigned int flags, unsigned int outfmt) | |
16446 | 72 { |
36309 | 73 int res; |
36757
18518cb8a99e
vf_screenshot: Fix memory leak on resolution/aspect changes.
reimar
parents:
36756
diff
changeset
|
74 if (vf->priv->ctx) sws_freeContext(vf->priv->ctx); |
16464
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
75 vf->priv->ctx=sws_getContextFromCmdLine(width, height, outfmt, |
27947 | 76 d_width, d_height, IMGFMT_RGB24); |
16446 | 77 |
36755 | 78 av_fast_malloc(&vf->priv->outbuffer, &vf->priv->outbuffer_size, d_width * d_height * 3 * 2); |
24731 | 79 vf->priv->avctx->width = d_width; |
80 vf->priv->avctx->height = d_height; | |
81 vf->priv->avctx->compression_level = 0; | |
16446 | 82 vf->priv->dw = d_width; |
83 vf->priv->dh = d_height; | |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
84 vf->priv->pic->linesize[0] = (3*vf->priv->dw+15)&~15; |
16446 | 85 |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
86 av_freep(&vf->priv->pic->data[0]); // probably reconfigured |
16446 | 87 |
36309 | 88 res = vf_next_config(vf,width,height,d_width,d_height,flags,outfmt); |
89 // Our draw_slice only works properly if the | |
90 // following filter can do slices. | |
91 vf->draw_slice=vf->next->draw_slice ? draw_slice : NULL; | |
92 return res; | |
16446 | 93 } |
94 | |
24731 | 95 static void write_png(struct vf_priv_s *priv) |
16446 | 96 { |
24731 | 97 char *fname = priv->fname; |
16446 | 98 FILE * fp; |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
99 AVPacket pkt; |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
100 int res, got_pkt; |
16446 | 101 |
102 fp = fopen (fname, "wb"); | |
103 if (fp == NULL) { | |
27947 | 104 mp_msg(MSGT_VFILTER,MSGL_ERR,"\nPNG Error opening %s for writing!\n", fname); |
105 return; | |
16446 | 106 } |
27947 | 107 |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
108 av_init_packet(&pkt); |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
109 pkt.data = priv->outbuffer; |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
110 pkt.size = priv->outbuffer_size; |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
111 res = avcodec_encode_video2(priv->avctx, &pkt, priv->pic, &got_pkt); |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
112 if (res >= 0 && got_pkt && pkt.size > 0) |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
113 fwrite(priv->outbuffer, pkt.size, 1, fp); |
16446 | 114 |
115 fclose (fp); | |
116 } | |
117 | |
118 static int fexists(char *fname) | |
119 { | |
120 struct stat dummy; | |
36750 | 121 return stat(fname, &dummy) == 0; |
16446 | 122 } |
123 | |
16450 | 124 static void gen_fname(struct vf_priv_s* priv) |
16446 | 125 { |
16450 | 126 do { |
36752
af146b0d33f0
Allow specifying custom path and name for screenshot files.
reimar
parents:
36751
diff
changeset
|
127 snprintf(priv->fname, sizeof(priv->fname), "%s%04d.png", priv->prefix, ++priv->frameno); |
16450 | 128 } while (fexists(priv->fname) && priv->frameno < 100000); |
16457 | 129 if (fexists(priv->fname)) { |
27947 | 130 priv->fname[0] = '\0'; |
131 return; | |
16457 | 132 } |
16450 | 133 |
134 mp_msg(MSGT_VFILTER,MSGL_INFO,"*** screenshot '%s' ***\n",priv->fname); | |
135 | |
136 } | |
137 | |
16464
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
138 static void scale_image(struct vf_priv_s* priv, mp_image_t *mpi) |
16450 | 139 { |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
140 if (!priv->pic->data[0]) |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
141 priv->pic->data[0] = av_malloc(priv->pic->linesize[0]*priv->dh); |
27947 | 142 |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
143 sws_scale(priv->ctx, mpi->planes, mpi->stride, 0, priv->dh, priv->pic->data, priv->pic->linesize); |
16450 | 144 } |
16446 | 145 |
30642
a972c1a4a012
cosmetics: Rename struct vf_instance_s --> vf_instance.
diego
parents:
30638
diff
changeset
|
146 static void start_slice(struct vf_instance *vf, mp_image_t *mpi) |
27948 | 147 { |
35114
bec9e755fb9b
Fix handling of out-of-order slice rendered frames.
reimar
parents:
34659
diff
changeset
|
148 mpi->priv= |
16464
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
149 vf->dmpi=vf_get_image(vf->next,mpi->imgfmt, |
27947 | 150 mpi->type, mpi->flags, mpi->width, mpi->height); |
16450 | 151 if (vf->priv->shot) { |
27947 | 152 vf->priv->store_slices = 1; |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
153 if (!vf->priv->pic->data[0]) |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
154 vf->priv->pic->data[0] = av_malloc(vf->priv->pic->linesize[0]*vf->priv->dh); |
16450 | 155 } |
27947 | 156 |
16450 | 157 } |
158 | |
30642
a972c1a4a012
cosmetics: Rename struct vf_instance_s --> vf_instance.
diego
parents:
30638
diff
changeset
|
159 static void get_image(struct vf_instance *vf, mp_image_t *mpi) |
27948 | 160 { |
16464
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
161 // FIXME: should vf.c really call get_image when using slices?? |
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
162 if (mpi->flags & MP_IMGFLAG_DRAW_CALLBACK) |
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
163 return; |
27947 | 164 vf->dmpi= vf_get_image(vf->next, mpi->imgfmt, |
165 mpi->type, mpi->flags/* | MP_IMGFLAG_READABLE*/, mpi->width, mpi->height); | |
16456 | 166 |
16450 | 167 mpi->planes[0]=vf->dmpi->planes[0]; |
168 mpi->stride[0]=vf->dmpi->stride[0]; | |
169 if(mpi->flags&MP_IMGFLAG_PLANAR){ | |
27947 | 170 mpi->planes[1]=vf->dmpi->planes[1]; |
171 mpi->planes[2]=vf->dmpi->planes[2]; | |
172 mpi->stride[1]=vf->dmpi->stride[1]; | |
173 mpi->stride[2]=vf->dmpi->stride[2]; | |
16450 | 174 } |
175 mpi->width=vf->dmpi->width; | |
176 | |
177 mpi->flags|=MP_IMGFLAG_DIRECT; | |
27947 | 178 |
16450 | 179 mpi->priv=(void*)vf->dmpi; |
16446 | 180 } |
181 | |
30642
a972c1a4a012
cosmetics: Rename struct vf_instance_s --> vf_instance.
diego
parents:
30638
diff
changeset
|
182 static int put_image(struct vf_instance *vf, mp_image_t *mpi, double pts) |
16446 | 183 { |
16464
0a828abf1f0f
Fix multiple issues: No picture at all, broken pictures, only every second
reimar
parents:
16457
diff
changeset
|
184 mp_image_t *dmpi = (mp_image_t *)mpi->priv; |
27947 | 185 |
35114
bec9e755fb9b
Fix handling of out-of-order slice rendered frames.
reimar
parents:
34659
diff
changeset
|
186 if(!(mpi->flags&(MP_IMGFLAG_DIRECT|MP_IMGFLAG_DRAW_CALLBACK))){ |
27947 | 187 dmpi=vf_get_image(vf->next,mpi->imgfmt, |
188 MP_IMGTYPE_EXPORT, 0, | |
189 mpi->width, mpi->height); | |
190 vf_clone_mpi_attributes(dmpi, mpi); | |
191 dmpi->planes[0]=mpi->planes[0]; | |
192 dmpi->planes[1]=mpi->planes[1]; | |
193 dmpi->planes[2]=mpi->planes[2]; | |
194 dmpi->stride[0]=mpi->stride[0]; | |
195 dmpi->stride[1]=mpi->stride[1]; | |
196 dmpi->stride[2]=mpi->stride[2]; | |
197 dmpi->width=mpi->width; | |
198 dmpi->height=mpi->height; | |
16450 | 199 } |
16446 | 200 |
201 if(vf->priv->shot) { | |
36756
6e78b4f9569f
vf_screenshot: Simplify repeat screenshot handling logic.
reimar
parents:
36755
diff
changeset
|
202 vf->priv->shot &= ~1; |
27947 | 203 gen_fname(vf->priv); |
204 if (vf->priv->fname[0]) { | |
205 if (!vf->priv->store_slices) | |
206 scale_image(vf->priv, dmpi); | |
207 write_png(vf->priv); | |
208 } | |
209 vf->priv->store_slices = 0; | |
16446 | 210 } |
211 | |
17906
20aca9baf5d8
passing pts through the filter layer (lets see if pts or cola comes out at the end)
michael
parents:
17012
diff
changeset
|
212 return vf_next_put_image(vf, dmpi, pts); |
16446 | 213 } |
214 | |
29702
28fb967d78d8
Mark some functions that do not need external visibility as static.
diego
parents:
29064
diff
changeset
|
215 static int control (vf_instance_t *vf, int request, void *data) |
16446 | 216 { |
19833 | 217 /** data contains an integer argument |
218 * 0: take screenshot with the next frame | |
219 * 1: take screenshots with each frame until the same command is given once again | |
220 **/ | |
16446 | 221 if(request==VFCTRL_SCREENSHOT) { |
27947 | 222 if (data && *(int*)data) { // repeated screenshot mode |
36756
6e78b4f9569f
vf_screenshot: Simplify repeat screenshot handling logic.
reimar
parents:
36755
diff
changeset
|
223 vf->priv->shot ^= 2; |
27947 | 224 } else { // single screenshot |
36756
6e78b4f9569f
vf_screenshot: Simplify repeat screenshot handling logic.
reimar
parents:
36755
diff
changeset
|
225 vf->priv->shot |= 1; |
27947 | 226 } |
16446 | 227 return CONTROL_TRUE; |
228 } | |
229 return vf_next_control (vf, request, data); | |
230 } | |
231 | |
232 | |
233 //===========================================================================// | |
234 | |
30642
a972c1a4a012
cosmetics: Rename struct vf_instance_s --> vf_instance.
diego
parents:
30638
diff
changeset
|
235 static int query_format(struct vf_instance *vf, unsigned int fmt) |
16446 | 236 { |
237 switch(fmt){ | |
238 case IMGFMT_YV12: | |
239 case IMGFMT_I420: | |
240 case IMGFMT_IYUV: | |
241 case IMGFMT_UYVY: | |
242 case IMGFMT_YUY2: | |
243 case IMGFMT_BGR32: | |
244 case IMGFMT_BGR24: | |
245 case IMGFMT_BGR16: | |
246 case IMGFMT_BGR15: | |
31082
92f88bb315c5
Add support for 12-bit color mode on framebuffer devices.
cehoyos
parents:
31003
diff
changeset
|
247 case IMGFMT_BGR12: |
16446 | 248 case IMGFMT_RGB32: |
249 case IMGFMT_RGB24: | |
27947 | 250 case IMGFMT_Y800: |
251 case IMGFMT_Y8: | |
252 case IMGFMT_YVU9: | |
253 case IMGFMT_IF09: | |
254 case IMGFMT_444P: | |
255 case IMGFMT_422P: | |
256 case IMGFMT_411P: | |
257 return vf_next_query_format(vf, fmt); | |
16446 | 258 } |
259 return 0; | |
260 } | |
261 | |
27949
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
262 static void uninit(vf_instance_t *vf) |
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
263 { |
30143 | 264 avcodec_close(vf->priv->avctx); |
27949
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
265 av_freep(&vf->priv->avctx); |
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
266 if(vf->priv->ctx) sws_freeContext(vf->priv->ctx); |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
267 av_freep(&vf->priv->pic->data[0]); |
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
268 av_frame_free(&vf->priv->pic); |
36755 | 269 av_freep(&vf->priv->outbuffer); |
36752
af146b0d33f0
Allow specifying custom path and name for screenshot files.
reimar
parents:
36751
diff
changeset
|
270 free(vf->priv->prefix); |
27949
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
271 free(vf->priv); |
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
272 } |
88d793e52bd9
cosmetics: Move up uninit() to avoid a forward declaration.
diego
parents:
27948
diff
changeset
|
273 |
30638
a7b908875c14
Rename open() vf initialization function to vf_open().
diego
parents:
30421
diff
changeset
|
274 static int vf_open(vf_instance_t *vf, char *args) |
16446 | 275 { |
276 vf->config=config; | |
277 vf->control=control; | |
278 vf->put_image=put_image; | |
279 vf->query_format=query_format; | |
16450 | 280 vf->start_slice=start_slice; |
281 vf->draw_slice=draw_slice; | |
282 vf->get_image=get_image; | |
16874 | 283 vf->uninit=uninit; |
36754 | 284 vf->priv = calloc(1, sizeof(struct vf_priv_s)); |
36753
b3afe52119c4
vf_screenshot: Switch to avcodec_encode_video2 API.
reimar
parents:
36752
diff
changeset
|
285 vf->priv->pic = av_frame_alloc(); |
36752
af146b0d33f0
Allow specifying custom path and name for screenshot files.
reimar
parents:
36751
diff
changeset
|
286 vf->priv->prefix = strdup(args ? args : "shot"); |
34566
f3d53cd55376
Update deprecated avcodec_alloc_context()/avcodec_open() API calls
siretart
parents:
32537
diff
changeset
|
287 vf->priv->avctx = avcodec_alloc_context3(NULL); |
34659 | 288 vf->priv->avctx->pix_fmt = PIX_FMT_RGB24; |
24731 | 289 avcodec_register_all(); |
35715
8517826b0dbd
Replace CODEC_IDs their modern AV_-prefixed counterparts.
diego
parents:
35712
diff
changeset
|
290 if (avcodec_open2(vf->priv->avctx, avcodec_find_encoder(AV_CODEC_ID_PNG), NULL)) { |
24731 | 291 mp_msg(MSGT_VFILTER, MSGL_FATAL, "Could not open libavcodec PNG encoder\n"); |
292 return 0; | |
293 } | |
16446 | 294 return 1; |
295 } | |
296 | |
297 | |
25221 | 298 const vf_info_t vf_info_screenshot = { |
16446 | 299 "screenshot to file", |
300 "screenshot", | |
301 "A'rpi, Jindrich Makovicka", | |
302 "", | |
30638
a7b908875c14
Rename open() vf initialization function to vf_open().
diego
parents:
30421
diff
changeset
|
303 vf_open, |
16446 | 304 NULL |
305 }; | |
306 | |
307 //===========================================================================// |