fixup! move __IsReady function into the critical section

hotfix for webtct failure

Change-Id: I63d9ed1bedda30f7fe065c3f819118b423a8b929

Changes rand() to rand_r()

SVACE warned that rand makes no guaranteee of thread safety.

[Version] 0.5.33
[Issue Type] Bug

Change-Id: I256cf902b08a7f4fd334e79ad9c249b323f31c57
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>

CAudioError: revise exception string format and the concatenate logic

[Version] 0.5.32
[Issue Type] Revise

Change-Id: I5ec5324f72d838b19a4d39387d82bcaa8ccfca3b

CAudioError: Fix bug of 'Exception objects must be nothrow copy constructible'

https://wiki.sei.cmu.edu/confluence/display/cplusplus/ERR60-CPP.+Exception+objects+must+be+nothrow+copy+constructible

[Version] 0.5.31
[Issue Type] Revise

Change-Id: I31225a025723dd2581f1c941012999143a9f06ee

move __IsReady function into the critical section

IsReady() was moved into critical section to support thread-safe.
stress testcase was added.

[Version] 0.5.30
[Issue Type] DF200907-00219

Change-Id: I45811c183636faabfd66090b96ddfa910f733496
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>

fixup! Add writable-size wait timeout log / Terminate write operation with accumulated timeouts

[Version] 0.5.29
[Issue Type] Build

Change-Id: I02ed186a776a39703a89196c7c53201539fbff04

Add writable-size wait timeout log / Terminate write operation with accumulated timeouts

[Version] 0.5.28
[Issue Type] Enhancement

Change-Id: Ie1f43daefd63bace231e048891197258f22c10e6

Revise dummy stream write function using anonymous vector objects

In addition, remove explicit c++ mode flag

[Version] 0.5.27
[Issue Type] Revise

Change-Id: I81e4481e9030d1dda006947e9a36884ba0f6304a

use auto for constexpr whenever possible

Change-Id: I0afca45754dc2487ed23d7bb63f2c5293f426f46

Revise cpp codes (mutex/cond)

use std::mutex instead of pthread_mutex for internal locking.
use lock_guard/unique_lock whenever possible, which is convenient RAII-style mechanism for owning a mutex.
use std::condition_variable instead of pthread_cond for internal wait/signal

[Version] 0.5.26
[Issue Type] Revise

Change-Id: I2aee6405c173c5df2b391998d8e483e1c528c453

Add audio-in set/get volume

Change-Id: I281840bb01528f2903430fcd036a3c4812aec68c

Move drain method to subclass CAudioOutput

fix improper virtual function keyword

[Version] 0.5.25
[Issue Type] Revise

Change-Id: I038e0450c8342dcf0155b12df1a98e2ec981a7c9

fixup! Avoid infinite blocking on internalWait() by replacing cond_wait with cond_timedwait

[Version] 0.5.24
[Issue Type] Bug

Change-Id: Ic116edc1ee4a23c2c6a3c5bd3cf4ae96e414b3b8

Fix improper implementation of CPulseAudioClient::getReadableSize()

+ Fix 'Dereference null return value' issue reported by Covertity

[Version] 0.5.23
[Issue Type] Bug

Change-Id: Ic6ad0f52b4ee1ad7e29a8c6f6bdcd46e1f6b734d

Avoid infinite blocking on internalWait() by replacing cond_wait with cond_timedwait

[Version] 0.5.22
[Issue Type] Deadlock

Change-Id: If7064ca734a93a8c75b88b18298f5868feffdfad

fixup! Change reference link in doc file

The guideline has been changed.

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

Change reference link in doc file

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

Update coverage exception macros

Change-Id: I265dcfdf90fab8960ebbb8a718834738b72d1c59

use constexpr whenever possible / remove unused member function

Change-Id: If9726261b3603644c9cf2f111888bd86df5f38fb

Revise CAudioError to derive std::exception

catch error by const reference

Change-Id: Ied91d1ea47de3e899466a6a23b787729ad13db81

Add deprecated symbol related to Unity for compatibility

 - audio_out_create()
 - audio_out_set_interrupted_cb()

[Version] 0.5.21
[Issue Type] Compatibility

Change-Id: I76a7619afe6e88e8cdcb3926e8a2b4fbff497039

revise CAudioInfo conversion

Change-Id: I0116ac94c3444179292f4b668fa35ae4f590b039

Remove unused internal error enums

Following enums are removed.
  EError::ERROR_INVALID_SAMPLERATE,
  EError::ERROR_INVALID_CHANNEL,
  EError::ERROR_INVALID_FORMAT

[Version] 0.5.20
[Issue Type] Refactoring

Change-Id: Iace839139ad71177689fb1269f0ec90341bfd771

revise conversion utility functions

Change-Id: I503db61756e851be77558e147d3e4ade68f5e867

revise callback assignments

Change-Id: I4fa2dfe77af2d0412f97a737633eb599a98b3b1e

Revise cpp codes

  - remove {} for single statement.
  - use noexcept keyword for non-throw methods.
  - use override keyword for overrided function.
  - remove unused methods.
  - use default keyword for empty destructors.
  - use explict keyword on single argument class constructor to prevent auto conversion.
  - use nullptr instead of NULL if possible.
  - use ! operator instead of comparing with false.
  - use auto when initializing with a cast to avoid duplicating the type name.
  - remove explict end return of void functions.

[Version] 0.5.19
[Issue Type] Refactoring

Change-Id: I5118a6ff5e85f9d9e02c7875b2cbe691652ef064

Revise header doxygen descriptions

[Version] 0.5.18
[Issue Type] Doxygen

Change-Id: Ifdd05b2b7ed64c98af1a842e774f48fd56daf8b4

[ACR-1434] Extend channel and format parameters

- support multi-channels (3~8) for recording only
- support S32LE format type

Change-Id: I5b31a7db0cfecec1d118621ed1ef4014805b6334

Add gcov package for line coverage automation

[Version] 0.5.17
[Issue Type] Coverage

Change-Id: I545d8b9ceb0e051409a07bc8308aafda6274d19c

Merge branch 'tizen_line_coverage' into tizen

Change-Id: Ia162ed8e8ea4bb8d60ce2ae1333d5e5c232caa12

CAudioOutput: Apply high latency in case of network source media audio type

[Version] 0.5.16
[Issue Type] New feature

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

Merge branch 'tizen' into tizen_line_coverage

Change-Id: I2f9d5cbdb39ab3552e1472c27272e5cedf804983

Merge "CAudioInput/Output: Apply VoIP latency in case of VoIP audio type" into tizen

CAudioInput/Output: Apply VoIP latency in case of VoIP audio type

There was no implementation of setting specific latency value except default case.
An audio type can be set by audio_in[out]_set_sound_stream_info() and therefore
we can utilize the audio type to determine which latency type should be set.
If it is set, the audio core recoginizes it and set this stream with a specific
latency value.

VoIP type is one of the most common usecases for audio-io API. This case is
now added.

[Version] 0.5.15
[Issue Type] Enhancement

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

Fix invalid format string

[Version] 0.5.15
[Issue Type] Build

Change-Id: Ia7106706f2e349d64b859bed53a20d0e85cbec95

Add pa_threaded_mainloop_lock() in CPulseAudioClient::write()

[Version] 0.5.14
[Issue Type] Bug fix

Change-Id: I56747241d749c2434101106c4aa1144aad1dde07
Signed-off-by: Jungsup Lee <jungsup4.lee@samsung.com>

Add more excludes

Change-Id: I0bbcebec7119a3b9561ac2cede1f0eec772105cb

Merge branch 'tizen' into tizen_line_coverage

Change-Id: Ib35e0f83c76869e8471b4deed33f4f0aec3f8a72

Remove unnecessary input drain code

[Version] 0.5.13
[Issue Type] Revise

Change-Id: Ia11fbd008af6cedf9856b4dcf6c153a8dbf4e0c8

Exclude more coverage check which can't be covered by this module

As following errors can't be covered by audio-io testcase itself, exclude from coverage scope
- pthread mutex errors
- sound_manager errors
- pulseaudio errors except policy restriction
- constructors which is not in use

Change-Id: I58d02dc65c16419b2c7eee95670f169327093109

Merge branch 'tizen' into tizen_line_coverage

Change-Id: I35670c14b02e01718d77f9c60f8d60d9eb6a5d61

Seperates privilege implementations from CAudioInput

[Version] 0.5.12
[Issue Type] Revise

Change-Id: I174b9cf04881acb0ff03fec1968e2f50aca8ec56

Remove unnecessary dependancies

[Version] 0.5.11
[Issue Type] Cleanup

Change-Id: Iee1173184d7eee42fcc8ff73418c52a2d586a55d

Patch for coverage

Change-Id: I0fbd522e8329acc848ab7dbf0abd482809b01999

Add gcov environment

Signed-off-by: SeokHoon Lee <andy.shlee@samsung.com>
Change-Id: I0c6d7329d49a8af7cd8a6c77840b2623e6f47fdd

fixup! fixup! Revise testsuite

 - fix 64bit build error

Change-Id: I810075a6bf0e70a0d608139c2044ce03d5dfc91f

fixup! Revise testsuite

Change-Id: I8bde941ab926ed9a8cc925d75688784dadc22b40

Revise testsuite

remove unnecessary glib header include

[Version] 0.5.10
[Issue Type] Revision

Change-Id: Ie232cd33c38b205a0cb1795c220a0265b46e199a

Support 24bit sample format and up-to 192kHz sample rate

[Version] 0.5.9
[Issue Type] Enhancement

Change-Id: I1f5dafa187781e60041ce6667cdc66c156466076

Skip drain when stream is not started yet

[Version] 0.5.8
[Issue Type] Bug Fix

Change-Id: I741348f3ed8fe082d6734dd91d8e79533af1c865

Remove 3.0 deprecated API

Please refer following deprecation policy.

https://developer.tizen.org/development/training/native-application/understanding-tizen-programming/api-versioning-and-deprecation-policy-tizen-platform

[Version] 0.5.7
[Issue Type] Deprecation

Change-Id: I79cf36662d10fdb7140f96bca8aef8f905cec89f

Fix for TV product build macro

[Version] 0.5.6
[Issue Type] Build

Change-Id: Ia0541ac27ef8686579b38d8a440d6847174d3236

Update doc for deprecation

[Version] 0.5.5
[Issue Type] Deprecation

Change-Id: Ib164beb96006c7b8fc44bf7436b6dc18f63ea3b2

Remove 3.0 deprecated API implementation

[Version] 0.5.4
[Issue Type] Deprecation

Change-Id: I8acb7365e55e0d76afe8fdca9e559e0b90f5743f

Throw invalid operation error when unprepare() is called at inside of pulse-thread

seperate tool package for containing test binary

[Version] 0.5.3
[Issue Type] Bug Fix

Change-Id: If5861e2d217d52da7dd9162aeeb74bb0498e3059

Add session mutex lock

- revise try-catch blocks / single line statement
- add more capi layer logs for easy debugging

[Version] 0.5.2
[Issue Type] Bug Fix

Change-Id: Ide35f778da97fe9d69227628d6d97ed607509aec

Revise CPulseAudioClient::finalize() to free all the resources which have been already used

A failure can be happened during initialize(), we do not lean on the __mIsInit member variable
which is only set on the last stage of the initialize() function.

[Version] 0.5.1
[Issue Type] Bug Fix

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

Disable deprecated sound-manager internal API call

Other deprecated code will be removed soon

[Version] 0.5.0
[Issue Type] Deprecation

Change-Id: I60b2d06cd51ffb391083b843630498dd1b24f424

Enable dummy write on streamCb only for async mode backward compatibility

set started/underflow for playback debugging purpose

[Version] 0.4.5
[Issue Type] Bug Fix

Change-Id: I1c39d76895f93c6a6719d719c3382342f84be4d8
(cherry picked from commit 2f28d6f49ca602993014f849835865757e1f02b9)

Fix build warning

[Version] 0.4.4
[Issue Type] Build warning

Change-Id: I50e90f6c0cfcc2280ce0a98490d66b3355306c90
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit acc37bef636f7e140cb84b3b3366948fd15afbda)

Add to lock/unlock focus Id mutex

There are cases that focus or focus watch callback is used internally
for session backward compatibility. Because there has not been any mutex for
focus id, sometimes an unintended error happens accessed from another thread.
Now, a mutex for the focus id is added and operated to secure the critical section.

[Version] 0.4.3
[Issue Type] Bug fix

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

Rearrange internalLock()

'internalLock' is for protecting mpPulseAudioClient handle.
But in some functions, it is used too widely it causes dead-lock
issue with internal focus callback for session backward compatibility.

Original patch
 : https://review.tizen.org/gerrit/#/c/153596/

[Version] 0.4.2
[Issue Type] Bug fix

Change-Id: I5c4633e3a9ecfa631bd860a85edf8511cd05c28c
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit f3f84b6d2c5257f2a25d710a336a0a52ddd42b78)

Apply blocking policy in case of prepare() functions

If audio io handle uses media session with MIX_WITH_OTHERS option and other process
has already acquired a focus which has more higher priority than media, policy error
will be returned from prepare() functions.
 - audio_out_prepare(), audio_in_prepare()

Unnecessary session backward compatibility codes are also removed in some functions
exported since tizen 3.0.
 - audio_out_pause(), audio_in_pause()
 - audio_out_resume(), audio_in_resume()

Original patch is
 : https://review.tizen.org/gerrit/#/c/125413/

[Version] 0.4.1
[Profile] Common
[Issue Type] Backward compatibility

Change-Id: Id0710c2847903cf96cdce38eab909b5c2e103c5f
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit 6035bcf1a578fa1c0533a7d850e0e9f2794a46ab)

Modification for Tizen C++ Coding Rule Checker

[Version] 0.4.0
[Issue Type] Cleanup

Change-Id: I1e94771fa0f3ebc5cf6fce8cce0b15deacfb5abf

Merge from mobile product code for backward compatibility

(common)
 - seperate cond wait mutex
 - reject read on interrupted situtation
 - no auto-resume by audio-io
 - async send interrupt completed event using g_idle_add()

(except TV product)
 - seperate async playback latency managment
 - start stream as corked
 - dummy write on start

[Version] 0.3.81
[Issue Type] Compatibility

Change-Id: Ie55c7c723433db0c607e08319a09429d163c6b78

Add mutex for internal focus cb

[Version] 0.3.80
[Issue Type] Enhancement

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

Revise exception handling

- Use reference variable on catching exception
- Remove throw specification which mostly useless
- Add few more null checks
- Revise re-throw method
- Fix SVACE defects (WGID:13946)
- Add copy constructor for CAudioError class

[Version] 0.3.79
[Profile] Common
[Issue Type] Enhancement

Change-Id: Iadf4fcceaf0f4c07bf575f2809e16ab0cb60608d

Support VOICE_RECOGNITION_SERVICE stream type

[Version] 0.3.78
[Profile] Common
[Issue Type] Enhancement

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

Fix exception handling of 'new' operator.

If it fails memory allocation by calling operator 'new', 'std:bad_alloc' exception will throw.
Therefore, never check null-pointer at next line. These situations cause potential leak also.
This patch uses exception handling of 'new' operator instead of checking null-pointer.

[Version] 0.3.77
[Profile] Common
[Issue Type] Bug

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: Ibdd9c4d93273d1a0874eee4cdb5056ff5ae35407

Fix build error due to -Wformat

[Version] 0.3.77
[Profile] Common
[Issue Type] Enhancement

Change-Id: I5b5b9dd123eeeb5445b108063ddb158e0597a1d5

Manage current acquired focus state and check it before acquiring/releasing focus

[Version] 0.3.76
[Profile] Common
[Issue Type] Backward compatibility

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

Use static_cast instead of dynamic cast

[Version] 0.3.75
[Profile] Common
[Issue Type] Enhancement

Change-Id: I5eb19a26aed1d01cebb77f33c8376a10ddb23a2d

Fix convertInterruptedCode()

[Version] 0.3.74
[Profile] Common
[Issue Type] Backward compatibility

Change-Id: I2fbe48dcf447089ed1096ce750bb7db75d4f90e5
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit 54c7a443738deaf9fa3421b8372ed071fa457a76)

Fix doxygen comment which is different with actual implementation

Change-Id: Ie47763f3b1b28d72d7017cf315bdf659fc57ef84
(cherry picked from commit 6708973408ad4b2aa0fec759824ef15e09549d1a)

Fix reference link

[Version] 0.3.73
[Profile] Common
[Issue Type] Documentation

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

Merge branch 'tizen_3.0' into tizen

Change-Id: Ia1c6ddf672ed79f00335455cc4cc98853f56c115

Enhance backward compatibility for media session

Pass no-resume option when acquiring/releasing focus internally in case of media session type.
Set reacquisition to false in case of media session type without resume setting.
Those are added to meet the previous media session behavior.

[Version] 0.3.72
[Profile] Common
[Issue Type] Backward compatibility

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

Skip drain() function when onInterrupt is invoked

These codes are disabled temporarily due to the blocking of the drain() function in onInterrupt.

[Version] 0.3.71
[Profile] Common
[Issue Type] Fix bug

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

Abandon internal focus if it exists when setStreamInfo() is called

[Version] 0.3.70
[Profile] Common
[Issue Type] Enhancement

Change-Id: I88bea331681cbb64a74b43d6a823807622892bed
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit 4098ee97917ff77c60236be096cbbf5b451dccd0)

Abandon internal focus if it exists when setStreamInfo() is called

[Version] 0.3.70
[Profile] Common
[Issue Type] Enhancement

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

Merge branch 'tizen_3.0' into tizen

Change-Id: I139d1245111fd7680078ea529dfca90879bad97e

Lock threaded mainloop before reset callbacks for safety

[Version] 0.3.69
[Profile] Common
[Issue Type] Bug

Change-Id: I258490b69489c09ac4fe17a388bbaf69b00a073c

Write dummy write if stream is not written at first callback during prepare situtaion

[Version] 0.3.68
[Profile] Common
[Issue Type] Bug

Change-Id: I3118c91af39d6022db9d4fb889a1a437a47dc605

Merge branch 'tizen_3.0' into tizen

Change-Id: I100336b4b3032b59e3488af71a3c1990eb20134f

Fix SVACE defects

[Version] 0.3.67
[Profile] Common
[Issue Type] Security

Change-Id: Ieaf38e106ca40196571cdfca35ccecad2b146dab

Avoid deadlock situation between onInterrupt() and unprepare()

[Version] 0.3.66
[Profile] Common
[Issue Type] Bug fix

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

Refactor codes of setting stream info. and add support for setting internal stream info. in case of VoIP session

[Version] 0.3.65
[Profile] Common
[Issue Type] Refactorying and backward compatibility

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

sched_yield() after sync read/write to avoid thread starvation

[Version] 0.3.64
[Profile] Common
[Issue Type] BUG

Change-Id: Iaa265baa7d49686c03ab8558277b5f9827ef673e
Signed-off-by: howon kim <howon.kim@samsung.com>

Implementation of isSkipSession() in CAudioSessionHandler.c

[Version] 0.3.63
[Profile] Common
[Issue Type] Backward compatibility

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

Merge branch 'tizen_3.0' into tizen

Change-Id: Iecf911453f488e7370484320e73376835f4940a9

Fix bug regarding ignore session

In case of default session(not registering internal focus handle), ignore session API does not work well.
Now, the invalid condition has been fixed and finalizing audio session handler when ignore session API has been added.

[Version] 0.3.62
[Profile] Common
[Issue Type] Bug fix

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

Cleanup code

[Version] 0.3.61
[Profile] Common
[Issue Type] Cleanup

Change-Id: I9d45df442e268d33c02bd6acefa94265138f03f9

Fix memory leak : valgrind

[Version] 0.3.60
[Profile] Common
[Issue Type] Memory Leak

Change-Id: Ia45b3bb108c57f0ccdc60441cd11a23a3213846b

Disable focus callback for session backward compatibility temporarily if tizen tv product, change log level

[Version] 0.3.59
[Profile] Common
[Issue Type] Backward compatibility

Change-Id: I20feb6baeaca7a4774431b81976b10f451505a78

Fix SVACE defects

[Version] 0.3.58
[Profile] Common
[Issue Type] SVACE

Change-Id: I20ae633c2d888e735bb286800860512ef7416679

Merge branch 'tizen_3.0' into tizen

Change-Id: Idfca420924e5aaacc52f344626c8d32f6ed01315

Fix drain called in thread and wait for drain completion

[Version] 0.3.57
[Profile] Common
[Issue Type] Bug

Change-Id: I308159d4bec56f4b1ccbbab9e5bd929984859955

audio-io fixed no callback issue at start

Changes:
 * In case of asyncronous write, never callback will be invoked when
   application has wrote less than prebuf(-1) at first callback.
   To avoid deadlock between applicaiton and pulseaudio, audio-io
   ensure to write at least size of prebuf.
 * Need to move enabling __mIsInit flag to state changed callback.
   Because of some audio-io APIs rejected at first stream callback.
   (e.g audio_out_drain, audio_out_flush, ...)

[Version] 0.3.56
[Profile] Common
[Issue Type] Compatibility

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: I64e43913c6a22ab2003615b40b9e616c59423628

Reset __mpEventListener member in finalize

[Version] 0.3.55
[Profile] Common
[Issue Type] Fix Bug (seg.fault)

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

Remove old temporal APIs

[Version] 0.3.54
[Profile] Common
[Issue Type] Cleanup

Change-Id: Iba934c17e113374a4fdb3bb1622883e32f54ba72