Do not store handle before it is initialised

handle is only set by ::addTimer so do not store its value by copy
before it is actually set.

Signed-off-by: Philip Lorenz <philip.lorenz@bmw-carit.de>

1 files changed, 3 insertions, 2 deletions

diff --git a/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp b/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp
index 46aa004..fbd4d4a 100644
--- a/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp
+++ b/AudioManagerUtilities/src/CAmCommonAPIWrapper.cpp
@@ -225,12 +225,13 @@ void CAmCommonAPIWrapper::registerTimeout(Timeout* timeout, const DispatchPriori
 
     //prepare handle and callback. new is eval, but there is no other choice because we need the pointer!
     sh_timerHandle_t handle;
-    timerHandles myHandle({handle,timeout});
-    mpListTimerhandles.push_back(myHandle);
 
     //add the timer to the pollLoop
     mpSocketHandler->addTimer(pollTimeout, &pCommonTimerCallback, handle, timeout);
 
+    timerHandles myHandle({handle,timeout});
+    mpListTimerhandles.push_back(myHandle);
+
     return;
 }