diff options
author | Patrik Flykt <patrik.flykt@linux.intel.com> | 2012-09-07 11:00:56 +0300 |
---|---|---|
committer | Patrik Flykt <patrik.flykt@linux.intel.com> | 2012-09-11 12:39:20 +0300 |
commit | d5a98f2fd1970aec2ea73c95da73d9ecd0c15d33 (patch) | |
tree | a887511bcd6f7661a299e418474fd3b7bad67149 /gweb/gweb.c | |
parent | 434d46d978c3d5b99b4856552ee5ab56eb156424 (diff) | |
download | connman-d5a98f2fd1970aec2ea73c95da73d9ecd0c15d33.tar.gz connman-d5a98f2fd1970aec2ea73c95da73d9ecd0c15d33.tar.bz2 connman-d5a98f2fd1970aec2ea73c95da73d9ecd0c15d33.zip |
gweb: Don't use debug functionality after possible free
When the callback has been called, the whole structure might be
freed. Thus don't call the debug function tied to the structure.
Diffstat (limited to 'gweb/gweb.c')
-rw-r--r-- | gweb/gweb.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/gweb/gweb.c b/gweb/gweb.c index 27ed6347..4c2f95c1 100644 --- a/gweb/gweb.c +++ b/gweb/gweb.c @@ -434,7 +434,6 @@ gboolean g_web_get_close_connection(GWeb *web) static inline void call_result_func(struct web_session *session, guint16 status) { - gboolean result; if (session->result_func == NULL) return; @@ -442,10 +441,8 @@ static inline void call_result_func(struct web_session *session, guint16 status) if (status != 0) session->result.status = status; - result = session->result_func(&session->result, session->user_data); + session->result_func(&session->result, session->user_data); - debug(session->web, "[result function] %s", - result == TRUE ? "continue" : "stop"); } static inline void call_route_func(struct web_session *session) |