Use av_strdup to allocate string, prevents the warning:
authorPatrik Kullman <patrik@yes.nu>
Sat, 21 Feb 2009 20:35:58 +0000 (20:35 +0000)
committerDiego Biurrun <diego@biurrun.de>
Sat, 21 Feb 2009 20:35:58 +0000 (20:35 +0000)
vhook/fish.c:130: warning: assignment discards qualifiers from pointer target type
patch by Patrik Kullman, patrik yes nu

Originally committed as revision 17494 to svn://svn.ffmpeg.org/ffmpeg/trunk

vhook/fish.c

index bd05b987b4add81128a75eb224b0dcf83d607fc4..3806df32eba074159ecd1f02283463e6c418ddbf 100644 (file)
@@ -127,7 +127,7 @@ int Configure(void **ctxp, int argc, char *argv[])
 
     optind = 1;
 
-    ci->dir = "/tmp";
+    ci->dir = av_strdup("/tmp");
     ci->threshold = 100;
     ci->file_limit = 100;
     ci->min_interval = 1000000;
@@ -368,7 +368,8 @@ void Process(void *ctx, AVPicture *picture, enum PixelFormat pix_fmt, int width,
                     f = fopen(fname, "w");
                     if (f) {
                         fprintf(f, "P6 %d %d 255\n", width, height);
-                        fwrite(buf, width * height * 3, 1, f);
+                        if (!fwrite(buf, width * height * 3, 1, f))
+                            av_log(ctx, AV_LOG_ERROR, "Couldn't write to PPM file %s\n", fname);
                         fclose(f);
                     }