auto_zoom: fix coverity issue 79/317479/2
authorInki Dae <inki.dae@samsung.com>
Wed, 11 Sep 2024 01:34:00 +0000 (10:34 +0900)
committerInki Dae <inki.dae@samsung.com>
Wed, 11 Sep 2024 03:58:47 +0000 (12:58 +0900)
Fix coverity issue - CID 1789466 - Data race condition by making sure locking
_inputQ.size() access.

Change-Id: I89c18dba3abafc69ca242e3b8fa7774b1956b2e1
Signed-off-by: Inki Dae <inki.dae@samsung.com>
services/auto_zoom/src/AutoZoom.cpp

index 91a61cdb86bc1ad2f8d10f6a0b269a2aeec5e1e7..1d102fbf7bf1665aa50414ec378ff0e1b24f0d53 100644 (file)
@@ -208,6 +208,8 @@ bool AutoZoom::isKeyValid(std::string key)
 
 void AutoZoom::add_input(BaseDataType &input_data)
 {
+       std::lock_guard<std::mutex> lock(_inputMutex);
+
        if (_inputQ.size() > 0) {
                SINGLEO_LOGE("Only one input data is allowed.");
                throw InvalidOperation("Only one input data is allowed.");
@@ -218,8 +220,6 @@ void AutoZoom::add_input(BaseDataType &input_data)
                throw InvalidOperation("Only IMAGE and FILE types are allowed.");
        }
 
-       std::lock_guard<std::mutex> lock(_inputMutex);
-
        _inputQ.push(input_data.clone());
 }