Hi,

valgrind reports "conditional jump depends on uninitialized value"
at SessionTimer.cpp:163. I suspect this is the min_se instance variable.

The attached patch initializes the min_se instance variable.

Regards,
Emil


-- 
Emil Kroymann
VoIP Services Engineer

Email: [email protected]
Tel: +49-30-203899885
Mobile: +49-176-38389303

ISACO GmbH
Kurfürstenstraße 79
10787 Berlin
Germany

Amtsgericht Charlottenburg, HRB 112464B
Geschäftsführer: Daniel Frommherz

From 0000e3cbd6e1136de3c9d9927662be40fae15498 Mon Sep 17 00:00:00 2001
From: Emil Kroymann <[email protected]>
Date: Wed, 29 Jun 2011 14:40:38 +0200
Subject: [PATCH] fix use of uninitialized value reported by valgrind

---
 core/plug-in/session_timer/SessionTimer.cpp |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/core/plug-in/session_timer/SessionTimer.cpp b/core/plug-in/session_timer/SessionTimer.cpp
index 7952a6c..b3f8b76 100644
--- a/core/plug-in/session_timer/SessionTimer.cpp
+++ b/core/plug-in/session_timer/SessionTimer.cpp
@@ -52,6 +52,7 @@ AmSessionEventHandler* SessionTimerFactory::getHandler(AmSession* s)
 SessionTimer::SessionTimer(AmSession* s)
   :AmSessionEventHandler(),
    s(s),
+   min_se(0),
    session_interval(0), 
    session_refresher(refresh_remote),
    accept_501_reply(true)
-- 
1.7.2.5

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Sems mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/sems

Reply via email to