mlt_property_get_time(): get mutex before accessing self->types

Fixes Coverity CID 709356: Data race condition (MISSING_LOCK)
Accessing variable "self->types" (mlt_property_s.types) requires the mlt_property_s.mutex lock.
871                self->types |= mlt_prop_string;

mlt_producer_seek(): check that eof is not NULL

Fixes Coverity CID 709348: Dereference after null check (FORWARD_NULL)
Comparing "eof" to null implies that "eof" might be null.
 310        else if ( use_points && ( eof == NULL || !strcmp( eof, "pause" ) ) && position >= mlt_producer_get_playtime( self ) )
 311        {
 312                mlt_producer_set_speed( self, 0 );
 313                position = mlt_producer_get_playtime( self ) - 1;
 314        }
At conditional (3): "use_points" taking the true branch.
Passing null variable "eof" to function "strcmp", which dereferences it. (The dereference is assumed on the basis of the 'nonnull' parameter attribute.)
 315        else if ( use_points && !strcmp( eof, "loop" ) && position >= mlt_producer_get_playtime( self ) )

mlt_producer_new(): check return value from mlt_producer_init()

mlt_playlist_init(): check return values from mlt_producer_init() and calloc()

Fixes Coverity CID 709327: Unchecked return value (CHECKED_RETURN)
Calling function "mlt_producer_init" without checking return value (as is done elsewhere 17 out of 20 times).
No check of the return value of "mlt_producer_init(producer, self)".
  73                mlt_producer_init( producer, self );

mlt_frame_get_waveform(): handle memory allocation failure

Fixes Coverity CID 709347.

At conditional (1): "bitmap != NULL" taking the false branch.
CID 709347: Dereference after null check (FORWARD_NULL)
Comparing "bitmap" to null implies that "bitmap" might be null.
 802        if ( bitmap != NULL )
 803                memset( bitmap, 0, size );
 804        mlt_properties_set_data( properties, "waveform", bitmap, size, ( mlt_destructor )mlt_pool_release, NULL );
...
 826                        // Position buffer pointer using y coordinate, stride, and x coordinate
Assigning null: "p" = "bitmap + i / skip + displacement * w".
 827                        unsigned char *p = bitmap + i / skip + displacement * w;
...
 831                                if ( *pcm < 0 )
Dereferencing null variable "p".
 832                                        p[ w * k ] = ( k == 0 ) ? 0xFF : p[ w * k ] + gray;
 833                                else
Dereferencing null variable "p".
 834                                        p[ w * k ] = ( k == height ) ? 0xFF : p[ w * k ] + gray;

mlt_filter.c: fix possible buffer overflows

Fixes Coverity CID 709411: Copy into fixed size buffer (STRING_OVERFLOW)
You might overrun the 20 byte fixed-size string "name" by copying "unique_id" without checking the length.
257        strcat( name, unique_id );

and

CID 709412: Copy into fixed size buffer (STRING_OVERFLOW)
You might overrun the 20 byte fixed-size string "name" by copying "unique_id" without checking the length.
302        strcat( name, unique_id );

mlt_filter_new(): check return value from mlt_filter_init()

Fixes Coverity CID 709326: Unchecked return value (CHECKED_RETURN)
Calling function "mlt_filter_init" without checking return value (as is done elsewhere 7 out of 8 times).
No check of the return value of "mlt_filter_init(self, NULL)".
 78                mlt_filter_init( self, NULL );
 79        return self;

vdpau: reduce runtime detection cost

Don't try to dlopen() every time, once is enough.

Fix vdpau library paths

They have changed in Debian.

vdpau compilation fix for ffmpeg 0.11

In file included from producer_avformat.c:157:0:
vdpau.c: In function ‘vdpau_get_buffer’:
vdpau.c:162:10: error: ‘AVFrame’ has no member named ‘age’
vdpau.c:169:10: error: ‘AVFrame’ has no member named ‘age’

Merge pull request #4 from mcfrisk/coverity

A few Coverity fixes

mlt_consumer_start(): check return value from mlt_properties_get_int()

Fixes Coverity CID 709343: Division or modulo by zero (DIVIDE_BY_ZERO)
Division by expression "mlt_properties_get_int(properties, "frame_rate_num")" which may be zero has undefined behavior
On this path, function call "mlt_properties_get_int(properties, "frame_rate_num")" has return value of 0
 442        int frame_duration = 1000000 / mlt_properties_get_int( properties, "frame_rate_num" ) *
 443                        mlt_properties_get_int( properties, "frame_rate_den" );

mlt_consumer_start(): get mutex before accessing put_active

Fixes Coverity CID 709355: Data race condition (MISSING_LOCK)
Accessing variable "self->put_active" (mlt_consumer_s.put_active) requires the mlt_consumer_s.put_mutex lock.
 411        self->put_active = 1;

mlt_consumer_new(): handle return value from mlt_consumer_init()

If init() fails return NULL. Fixes Coverity CID 709325.

At conditional (1): "self != NULL" taking the true branch.
 339        if ( self != NULL )
CID 709325: Unchecked return value (CHECKED_RETURN)
Calling function "mlt_consumer_init" without checking return value (as is done elsewhere 10 out of 11 times).
No check of the return value of "mlt_consumer_init(self, NULL, profile)".
 340                mlt_consumer_init( self, NULL, profile );

mlt_consumer.c: watch out for null pointer

Fixes Coverity CID 709393: Dereference before null check (REVERSE_INULL)
Directly dereferencing pointer "profile".
 235                profile->sample_aspect_num = mlt_properties_get_int( properties, "sample_aspect_num" );
Dereferencing "profile" before a null check.
 236                if ( profile )

mlt_cache.c: watch out for null pointer

Fixes Coverity CID 709346: Dereference after null check (FORWARD_NULL)
Comparing "result" to null implies that "result" might be null.
449                if ( result && result->data )
450                        result->refcount++;
Dereferencing null variable "result".
451                mlt_log( NULL, MLT_LOG_DEBUG, "%s: get %d = %p, %p\n", __FUNCTION__, cache->count - 1, *hit, result->data );
452

mlt_cache.c: check for null pointer

Fixes CID 709392: Dereference before null check (REVERSE_INULL).

riff.cc: Fail if lseek() fails

Fixes Coverity CID 709362: Argument cannot be negative (NEGATIVE_RETURNS)
Function "lseek(this->fd, 0LL, 0)" returns a negative number.
Assigning: signed variable "pos" = "lseek".
...
"pos" is passed to a parameter that cannot be negative.
548                fail_if( lseek( fd, pos, SEEK_SET ) == ( off_t ) - 1 );

riff.cc: Initialize data in constructor

Fixes Coverity CID 709444: Uninitialized scalar field (UNINIT_CTOR)
Non-static class member ""length"" is not initialized in this constructor nor in any functions that it calls.
Non-static class member ""name"" is not initialized in this constructor nor in any functions that it calls.
Non-static class member ""offset"" is not initialized in this constructor nor in any functions that it calls.
Non-static class member ""parent"" is not initialized in this constructor nor in any functions that it calls.
Non-static class member ""type"" is not initialized in this constructor nor in any functions that it calls.
Non-static class member ""written"" is not initialized in this constructor nor in any functions that it calls.

fix crash when switching image formats with alpha

This happens when switching from image format with distinct alpha
channel (yuv422) to one with embedded alpha channel (rgb24a).

Reported-by: j-b-m

fix memory read error found by valgrind

fix memory leak and missing unlock (coverity-709377 & 709354)

fix possible divide by zero (coverity-709344)

remove dead code (coverity-709337)

remove dead code (coverity-709335 & 709336)

remove dead code (coverity-709334)

fix faulty image format tests (coverity-709330, 709331, & 709332)

fix memory leak (coverity-709387, 709388, & 709389)

fix crash on null pointer

reported by Mikko Rapeli

initialze with calloc instead of memberwise (future-proofing)

fix uninitialized memory in mlt_events

fix regression building on 32-bit

fix uninitialized memory (coverity-709434)

fix out-of-bounds read (coverity-709368)

fix memory leak (coverity-710866)

fix memory leak (coverity-710865)

fix memory leak (coverity-710864)

fix memory leak (coverity-710863)

fix uninitialized var (coverity-709434)

fix uninitialized var (coverity-709433)

fix uninitialized array (coverity-709432)

fix uninitialized vars (coverity-709430 & 709431)

fix memory leak (coverity-709391)

remove unused function (coverity-709390)

fix memory leak (coverity-709386)

fix file handle leak (coverity-709383)

fix memory leak (coverity-709382)

fix file handle leak (coverity-709381)

fix memory leak (coverity-709380)

add Service::get_profile() returns mlt_profile

fix memory leak (coverity-709379)

fix memory leak (coverity-709378)

fix memory leak (coverity-709376)

fix memory leak (coverity-709375)

fix out-of-bounds read (coverity-709368 & 709369)

improve compatibility to compile composite sse2 (macports-35243)

move mutex and deque initialization to producer init method

fix leak of dv packets

fix memleak if file fails to open

add define MELT_NOSDL to not use SDL_main()

This is helpful on OS X when using melt as a utility to prevent the
program from appearing in the Dock.

fix crash in avformat consumer when audio encoding fails

accept file:// prefix on MLT XML file

fix some LADSPA crashing on dlclose on some systems (3538363)

Merge pull request #3 from j-b-m/master

QImage module: fix Qt4 detection

add support for time string to playlist blanks

make alpha channel independent of lifetime of region frame

fix memory leak when using filter_only with region transition

improve readability

rename this to transition and frame

fix filters that need progress with region filter/transition

See for example, obscure filter, as used in demo/mlt_obscure.

fix race condition in region filter when parallel processing

based on patch from j-b-m

add support for audio scrubbing to audio-only consumers

change printfs in filter_jackrack to mlt_log_verbose

send jack silence on buffer underrun

This occurs with the sdl_preview or any of the audio-only consumers when
paused. Otherwise, jack just plays unset memory, which is usually
something worse than just noise.

fix loading xml file on windows

stop checking PTS if we are only using DTS

Fix Qt4 detection when Qt3 is installed

add mlt_producer_seek_time and mlt_producer_frame_time

add to .gitignore

add const-ness to some strings in specialized service classes

fix loading xml with producer not enclosed in playlist or multitrack

fix clang errors

Merge branch 'master' of github.com:mltframework/mlt

add support for AUDIODEV environment var to rtaudio

overhaul a/v sync and seeking in avformat producer

The new_seek property changed to use_pts. This consolidates old seek and
new seek code, improves a/v sync for more files, and improves seek
performance for AVCHD in general (including libav).

fix BSTR string conversion under Windows

fix crash when reporting error in decklink producer init

set interim version to 0.8.1

update ChangeLog for v0.8.0

set version to 0.8.0

add release notes for v0.8.0

Merge branch 'master' of github.com:mltframework/mlt

fix image format regression in avformat caching

remove unused alpha_cache member

fix video_delay when using new_seek (AVCHD)

fix a/v sync after recent change for faster AVCHD seeking

Revert "Revert "make speed of editing AVCHD tolerable""

This reverts commit c02b411377f471c5c768dfb5f83171964c6c273a.

fix regression in build on 32-bit linux gcc 4.6.1

Revert "make speed of editing AVCHD tolerable"

This reverts commit b17e3e9f4eadc0dc53ded8a14df2059bb642afb1.

fix image format regression in avformat caching