Remove useless parameter v2numOfFrames

Change-Id: I6967a9637c370a884247104b93a89ff8d2272b73

Unify duplicated ID3tag extracting code

Change-Id: Idde219a2231733c90e70cd55c90d6b6f9325ab86

Remove id3 version specific 'mmfile_string_convert_v224' function and unreachable code

Change-Id: Ib528a6811d8197b51bb990e655e07a06062e4b4f

Remove duplicated code and unused parameter pre_checked

Change-Id: I6118f7dabce399f4fd731ac4bede1fef4462de50

Remove unused parameter pAuthor

Change-Id: I0ce1a7838763f26576c048035322752c835f221e

Remove unreachable code

 - Fix svace WGID 425086

 Varaible 'pForamtCtx' can't be null because of
 'NULL == formatContext->privateFormatData' checker.

Change-Id: I6e18ffad655bb3de10a8379257e65d473226dbb3

Update test code for macro refactoring

Change-Id: I4bc0fd5b84d0f97fdf924722477bb505cd7bdeb5

Fix coverity issue

  - Argument cannot be negative

  'ftell' returns signed long and it can be negative(-1).
  Unsigned int 'file_size(size)' can get tainted value.
  So signed long for ftell has been added and check if it is below zero.

Change-Id: Ibe651c19465acc57b2e4e31b705644887c18a623

Fix coverity issue

  - Resource leak
  Variable 'err_attr_name' should be freed by caller, but it is not used and freed.
  So, remove unused(useless) 'err_attr_name'.

Change-Id: I201c216828a05a0e620799145960de925c9ce524

Fix wrong duration for mpeg-ts

 Generally, mpegts has wrong start time(positive large value).
 So duration do not use start time for mpeg-ts.

Change-Id: I97ddc31860bdeb489abe41f4fbc7e9b61d2572d5

Fix build error due to toolchain upgrade (gcc6->gcc9)

  error: output trucated before the last format character [-Werror=format-truncation=]

Change-Id: I9a0c7460e1afc81e46d14a4190c89a644943bf64

Remove unused functions

Change-Id: Ib7ccc1e9da5c9793d0b2169c502e3df82e79a947

Replace SAFE_STRLCPY to mmfile_strdup

Change-Id: I81a8687b35b6066a7c6454e0af8242ec357b4f20

Remove unused variables

 - descriptionLen
 - pDescription
 - pRating
 - bTagVer2Found
 - bGenreUTF16

Change-Id: Id0ae4c97533d7a466686eacaf35af919d157e9f1

Code Cleanup for duplicated code of getting text encoding type

 - Add new funtion to get text encoding type

Change-Id: I33d11fa4f59c52f25b20b221bfe02a8a0bb5231d

Code Cleanup for duplicated code of getting synclyric

Change-Id: I20ab23c9cb0b3ad99121397cc402695f7961ee12

Code Cleanup for 'break' and add function to count the number of streams

Change-Id: I48131561ecded342993344d7198915ed9b935a75

Change global variable '_mm_file_format_mimetype_table' to static

Change-Id: I66368de864551766a51133c30f4b35827a2167d2

Improve id3tag parsing for duplicated code(skip new line)

Change-Id: I4914f579dadb63713efc8c37cce61a73db5b3c32

Fix coverity issue (memory leak)

Change-Id: I2494d0d8543ab3dc4fa88cd4520e0bbd20c7397a

Remove unused header file (duplicated with mm_util.h)

Change-Id: I6fb8771a23f61f1ed7aaf38d6daacefab5776eb3

Bump version to 0.6.80

 - Since this version, TC test value change is necessary.

Change-Id: Icd42fc0f7c2cef10e8e134fa21e353bb8128226c

Improve 'mm_file_id3tag_parse_v224' function to reduce Cyclomatic Complexity

 - It needs below UTC changes.
  https://review.tizen.org/gerrit/#/c/test/tct/native/api/+/214107/

Change-Id: Ie9ab72aa8d663ff759d12f2219edfa07c32c2f4c

Update global variable to static

Change-Id: Iad7002eb2932136ead5609d767620cc704d8f005

Bump version to 0.6.79

Change-Id: I8e2d044f79fcb07933ec6637174ed6902863ce0d

Improve PIC/APIC parsing of ID3tag to reduce CC and DC

Change-Id: Ibb0edee2b18f6ee03863686a39da700d43e7c852

Improve functions to reduce Duplicate Code

Change-Id: Ifc5191726c11039a818ecda28aeba609c5c02042

Improve '__mmfile_get_frame' function to reduce Cyclomatic Complexity

Change-Id: Ia0335b309420605b39dadef3b63c114e94f2564d

Improve '_get_contents_info' function to reduce Cyclomatic Complexity

Change-Id: I9b86b5432904d9475bc28a4f7e18ad42a1e42ea5

[DF190903-00975] Add code to support mpeg audio version 2, layer 1

Change-Id: I86c355baa73bb57285764f02ed00b7de2593c537

Improve 'MMFileFormatIsValidXXX' function to reduce DC

Change-Id: Id9ba66477f0b8766db1b33efb8d09bbb805b3b51

Improve '__AvConvertIMelody2MIDI' function to reduce DC

Change-Id: Ic751574c6677a1520121f36560d7b6762d2c8928

Improve '_mmf_MalibNextChunk' function and remove unused code

 - change 'switch ~ case' to 'looping const table'
 - remove unused function
 - remove unuseful condition (null check)

Change-Id: I227f94c6da3f6563c2ff05d1815bfee004634fa7

Remove unneccessary null check

 - 'mm_attrs_set_string/data_by_name' can set null string/data
 - artwork & thumbnail should null check due to memcpy

Change-Id: I219c365925eb03ddaffdbefa96677c7eb956aac8

Add new function to add bracket at genre tag

Change-Id: I2f9c9da246d573c53e2f364fb750afbf52f4e020

Replace duplicate '__getMimeType' and 'getMimeType' to 'mmfile_util_get_mimetype'

Change-Id: Ie01859409d1b6044c093c2de47f1e4449cf7b2aa

Improve 'mmfile_format_read_tag_ffmpg' function

Change-Id: I2edf99b08d29631d0363e1ce3493004ad99bfa21

Update debug msg

Change-Id: I4ca482bb7afb313de76f79a4624c563c197c679d

Update pkg version

Change-Id: I0f336e2e2cb9c5eeab27469aa465974496e58b5f

Change strcasecmp() to g_ascii_strcasecmp() because strcasecmp is locale sensitive.

Change-Id: Iff51728241d047f0af6f4dc2addbeecd5f573899

Merge "Support TPOS (Part of a Set)" into tizen

Merge "Just add description for the smta" into tizen

Increase _RETRY_SEARCH_LIMIT to 250. some contents need search count this much

Change-Id: I6825cea3e9a0a2a895b2e13af78940889c602744

Support TPOS (Part of a Set)

The ‘Part of a set’ frame is a numeric string that describes which part of a set the audio came from.
This frame is used if the source described in the “TALB” frame is divided into several mediums,
e.g. a double CD. The value MAY be extended with a “/” character and a numeric string containing the total number of parts in the set. E.g. “1/2”.

Change-Id: Ic0613adda4b54834892e7592df34598b9d14be9b

Just add description for the smta

Change-Id: I661cd6fe8667e3716e1269574836af63a61ac263

Support smta value 0x02

Change-Id: I7dc1f10917a706e03447a97b5ef69d945064d664

Change the size and nmemb of fread() to check the return value

Change-Id: Icff42f9f037df9b44cd5ab07420dab1037747463

Add to check the return of fseek()

 - If fseek() is failed, -1 is returned and errno is set to indicate the errors.

Change-Id: I7174ca625da1026ebe07cb72fd97459d89c05d75

Add to check the file permission before mm_file access a file.

- error case
Metadata-extractor did not return Permission denied.

Change-Id: I5ef74a95dcca60e451d9c1e55aab67694f7f208e

Coverity issue fix

Change-Id: Id90eb786575e82376fd0535ea11fd0fadfcd5470

Fix format error in logs

Change-Id: I024c8d532c47c3a58aa13f4b6e5fa5c6a647d7a2

Fix build error without gtests

Change-Id: I7eb22626683c57234d272302228beb831093d434

Merge "Adding initial structure for unittest" into tizen

Adding initial structure for unittest

- It has only one testcases for libmedia_utils of unittest
- It would be enabled by gtest build with '--define "gtests 1"'
- Two test contents would be copyied to '/usr/bin' directory

Change-Id: I6988de63bfd3c2cea7aa8518bf83d700d09d38cd

Apply new attribute API of libmm-common

- Remove dependency of mm_attrs_private.h.
- Use new API set in mm_attrs.h instead of those.

Change-Id: Icd720cc02b677b881e4266c8a4a0cd79f72ab5ce
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

Remove dlog from Requires

Change-Id: Ie270508a787cc45064acfc7a3c5f1b17d658c8d2

Remove duplicate verification code

Use mmfile_free instead of FREE_EX
Remove duplicate null check

Change-Id: I73101a08098966b1e026d2d0c384632abc1c74d9
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

[PLM P180706-00145]Fix character set issue for some languages used in Europe. And svace issues

Change-Id: I90bdeb4d5ba89b0f130995bfe575766a24ce37bb

Add fortify, stack canary option

Change-Id: I52c4da8092619c3d0d165082bb8a9be64766fab0
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Implement Spherical video metadata V2 extraction in MKV/WebM

- No stereo-mode support added as far as it is stored using the
  existing StereoMode element specified in the Matroska spec. and
  can be supported in the Matroska format parser

- WebM files prepared with MKVMerge tool with "-w" option turned on
  (force WebM sub-format) are not recognized by libmm-fileinfo

- Ambisonic audio is not specified in Google's RFC v2

Change-Id: I4935602afb658bfa6971be247a4ffd1e3332fe55
Signed-off-by: Mykola Alieksieiev <m.alieksieie@samsung.com>

support webm format

Change-Id: I4154332ff696b0efc060ad58876ad4d892e1715c

add support ape format

Change-Id: I529ee542ea735c8d33ee1b4eefa15135e3c8df3c

Add debug msg for error case

Change-Id: If7c7e0ee7994ca99b864a35fb0b67024a3b220a5

Fix deadlock during extracting tags

Change-Id: I16fe8b3844ace9be1edce741de533b40362ab4d6
Signed-off-by: Vitaliy Cherepanov <v.cherepanov@samsung.com>

MPEGTS content parsing timeout set.

Signed-off-by: ji.yong.seo <ji.yong.seo@samsung.com>
Change-Id: If2a7d646255ca0d35761c3d5aabd4667877d6d76
(cherry picked from commit fea2250192199c4669c0c5974fc1e73f27ffee8e)

Support AIFF

Change-Id: Idc4fd8a1caf4dc337fae1be7adf04debb24df3ee

Fix Memory leak

Change-Id: I53f4585b12708aeeb7b41452f4b79ddfdbebed81

Bug fix of tag checking

Change-Id: I7be4676b170f561f0e73f9548d28b7c2d1292093

Coverity Issue Fix

Change-Id: I54eec93fc513692bde3df0030f2841d23fff0afe

Remove useless configure file

Change-Id: I67d823a239c6cb89769ae52c49e404e365c2c36f

Modify the search function

Use glib API instead of dirent API

Change-Id: I2dc135b0f53f915ad478609b6457aa3999140780
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

mpg audio clip can't play , fix check mpeg audio layer 1 header

Change-Id: I34fb96eddddab03039be81d627d45b8fea7bf414

Apply tizen coding rule

Change-Id: I8fa2ede20e3dbb42a76d1e0a374f010750cd0e94

update msg format for debugging and apply tizen coding rule

Change-Id: I08190f96959658e63844ab77ff5a4ea065c43a33

Add Spherical Video Metadata V2 support

Change-Id: I215cce72ac37719c8fbb62da5953e498452ded1f
Signed-off-by: Mykola Alieksieiev <m.alieksieie@samsung.com>

Add to check NULL of pbSetup

Change-Id: I5ab01613fdc188446677cd55632dbf036715e745
Signed-off-by: Jiyong Min <jiyong.min@samsung.com>

Apply libav 12.1

Change-Id: Ic93a7190a1be1d8deefd7e70cd0ba4984af34b82

Remove deprecated ffurl related code. Use own io_context instead of it

Change-Id: I26a8c473fb9d4e9aa29aec928cf248299c4c5bfd

Svace issue fix

Change-Id: I479294aeda58771c9646c03a3d216c621b58af11

Add test code for getting video frame

Change-Id: I25ae5287f48837aca7ed27f369cc9aecc353a826

Remove useless include

Change-Id: I80534df4da30b1453e49524a679f0c0eae5adcc6

[SATIZENVUL-914] Add verify for value range

Change-Id: I6ffdc7c29568471347d4f58841925096a31a66e2
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove __MMFILE_FFMPEG_V085__

Change-Id: Iaddf2c369576a63cbd4457008049725ba7541b45

Change parameter type of time_stamp. double -> int64_t

Change-Id: I81de7b6465611f4ba482b75412a2b992b8c3a02b

Remove too old __MMFILE_FFMPEG_V085__ and related code. because __MMFILE_FFMPEG_V085__ was created when FFMpeg upgraded to v1.0 to backup code

Change-Id: Ibc3e6e96f89b5c4e1d69ade2f7ef8a32f2b40aae

Remove too old __MMFILE_FFMPEG_V100__ and related code. because __MMFILE_FFMPEG_V100__ was created when FFMpeg upgrade to v1.0

Change-Id: I34f40f317d3b246ba83573b2200435cfcf595c63

Fix Old Pixel format to New one

Change-Id: Ic238f65e2b1775dce3e575065f404dd876bf2867

[SATIZENVUL-949] check memory allocation

1. Add allocation check
2. Remove code bundled with _SM_ONLY

Change-Id: Ic833adfdb30711797c3e005f9e5b842a2bda3752
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Add support for Spherical Video and Ambisonics Audio Metadata

* Spherical Video Metadata V1 in mp4 files
* Spatial audio metadata extraction from SA3D atom nested in mp4a atom
* Supported different ambisonics types, orders and formats

Change-Id: I0768ef0e8f6875c47d1b58b35e4cbea68dacaf52
Signed-off-by: Mykola Alieksieiev <m.alieksieie@samsung.com>

fix invalid format argument types for dlog

Change-Id: Ia611bd5fccd8027d6fc3610c41562f59266543ec

[RQ170606-00002] fix invalid format argument types for dlog

Change-Id: Ib3cb1ae54e9a94de9044515cb37b751a695b9648

Modify to use vulnerable function 'sscanf'

Change-Id: I82d34b900905e3dc487a7b22b995c5c05b609ec1
Signed-off-by: Jiyong Min <jiyong.min@samsung.com>
(cherry picked from commit 81d593deaba59483538a1f357b676aa805807304)

remove MMFileUtilGetMetaDataFromMP4() when extract content attributes. because MMFileUtilGetMetaDataFromMP4() is only extracting metadata(tag)

Change-Id: Iac932732e656ae2320b123a1628816b89f33c581

Just update comment for smta and cdis which in samsung specific moov container

Change-Id: I3435b1613c2fdf449969241ddb637965adb18503

Add to check junk data

 [Problem] It is too long to read mpeg header when it was broken.
- When read tag from the file with the junk data, read whole file to find tag
because of wrong offset of box.
If the offset of box was wrong, add to check the moov data has already found at first.
After reading moov data is done, it check junk data for 1000 times.

Change-Id: I7ff67e7076870c6af43a1f3a98565c2b5cea856c
Signed-off-by: jiyong.min <jiyong.min@samsung.com>
Conflicts:
packaging/libmm-fileinfo.spec

Change-Id: I4ad0f2ff2000a8b8a784665a852a1e6f767c74d3
Signed-off-by: jiyong.min <jiyong.min@samsung.com>

Add tool package of executable file(mm_file_test) to improve security

Change-Id: I1a72170872b8f7a04109a452cf60e897d962a593
Signed-off-by: jiyong.min <jiyong.min@samsung.com>
Conflicts:
packaging/libmm-fileinfo.spec

Change-Id: Idcbce52b7e02cca763c2358de373e8307440c8ee

Replace readdir_r to readdir to fix build error

 [Problem] After update tool chain to GCC6, 'readdir_r is deprecated' build error happen

'readdir_r' is deprecated because of below reasons.
 *  On systems where NAME_MAX is undefined, calling readdir_r() may be
    unsafe because the interface does not allow the caller to specify
    the length of the buffer used for the returned directory entry.

 *  On some systems, readdir_r() can't read directory entries with
    very long names.  When the glibc implementation encounters such a
    name, readdir_r() fails with the error ENAMETOOLONG after the
    final directory entry has been read.  On some other systems,
    readdir_r() may return a success status, but the returned d_name
    field may not be null terminated or may be truncated.

Change-Id: Iaeabf4b09ab77dfedbc161c725c27a39ec4325c2
Signed-off-by: jiyong.min <jiyong.min@samsung.com>

Remove unused static variable to fix build error

 [Problem] After update tool chain to GCC6, 'definded but not used' build error happen

Before GCC6 -Wunused-variable (enabled by -Wall) didn’t warn for unused static const variables in C.
This was because some old code had constructs like: static const char rcs_id[] = "$Id:...";.
But this old special use case is not very common anymore. And not warning about such unused variables was hiding real bugs.
So GCC6 introduces -Wunused-const-variable (enabled by -Wunused-variable for C, but not for C++).

Change-Id: I075937a718b0dbbd3cab28df75084581bed363ab
Signed-off-by: jiyong.min <jiyong.min@samsung.com>

code refactoring

Change-Id: I935050c1fffe4643d3ebfeb493d012d1c5e01f7f
Signed-off-by: ji.yong.seo <ji.yong.seo@samsung.com>

packaging: fix build error by Werror

Different semantics for inline functions

While -std=gnu89 employs the GNU89 inline semantics, -std=gnu11 uses the
C99 inline semantics. The C99 inline semantics requires that if a
function with external linkage is declared with inline function
specifier, it also has to be defined in the same translation unit (TU).
Consequently, GCC now warns if it sees a TU such as the following:

  inline int foo (void);
  This example now gives the following diagnostic:

  f.c:1:12: warning: inline function 'foo' declared but never defined
    inline int foo (void);

Change-Id: I01405d2b7820e32400d5ec193c073c7ce6e1a50a
Signed-off-by: jiyong.min <jiyong.min@samsung.com>