[code cleanup] Add to check hard return value for file_read

Change-Id: I329980a09b6be133d374376a6bbe7b833afcf271

[code cleanup] Remove duplicated buffer allocation & file read
and remove unnecessary code

Change-Id: I57ec5316e4bd26d14d83ded18007bb5d209c33cb

Seperate getting mp3 information into getting mp3_tag_info and mp3_stream_info

  - When 'mmfile_format_open' function is called, it read mp3 file
    to get mp3 information. Because scanning mp3 file call the function
    2~3 times, there was a loss in performance.
    So we changed the function call from open to read and
    seperated into two sub_function.

Change-Id: Ica77bed739c89e41e3643889e29ada6eadb7fd28

[code cleanup] Improved code readability for getting ID3Tag

Change-Id: I1be3ab6b95a7e0c75e349b6de365769ef8e3944e

Unify duplicated ID3v2header getting code

Change-Id: Ib875c1e2421632dfb69f864df037a9e2f82d6258

Remove redundant condition check code

Change-Id: I774056be4f064d0a6cc0f5160aba6f5cacc12fd9

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