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 | |
| 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.
| -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) |