diff options
| author | Gerald Combs <gerald@zing.org> | 2015-08-02 15:33:07 -0700 |
|---|---|---|
| committer | Gerald Combs <gerald@wireshark.org> | 2015-08-03 03:06:00 +0000 |
| commit | 9557c73f81efad68ce3961a5ec18de2d985c4bb1 (patch) | |
| tree | 528dab6173e9f59072f3b90d2d5c2fbdaf470023 /epan/wslua/wslua_listener.c | |
| parent | 08e80b1653383fcbb42538b97e46e78d152b4893 (diff) | |
| download | wireshark-9557c73f81efad68ce3961a5ec18de2d985c4bb1.tar.gz | |
Make sure per-packet tap callbacks return gbooleans.
The tap API changed the return type of per-packet listener callbacks
from int to gboolean back in 2009. Update a bunch of functions and some
documentation accordingly.
Change-Id: I79affe65db975caed3cc296a7e2985b7b9cdf4cc
Reviewed-on: https://code.wireshark.org/review/9853
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'epan/wslua/wslua_listener.c')
| -rw-r--r-- | epan/wslua/wslua_listener.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/wslua/wslua_listener.c b/epan/wslua/wslua_listener.c index 8711ab1e20..64ed0610ff 100644 --- a/epan/wslua/wslua_listener.c +++ b/epan/wslua/wslua_listener.c @@ -81,11 +81,11 @@ static int tap_packet_cb_error_handler(lua_State* L) { } -static int lua_tap_packet(void *tapdata, packet_info *pinfo, epan_dissect_t *edt, const void *data) { +static gboolean lua_tap_packet(void *tapdata, packet_info *pinfo, epan_dissect_t *edt, const void *data) { Listener tap = (Listener)tapdata; - int retval = 0; + gboolean retval = FALSE; - if (tap->packet_ref == LUA_NOREF) return 0; + if (tap->packet_ref == LUA_NOREF) return FALSE; lua_settop(tap->L,0); @@ -107,7 +107,7 @@ static int lua_tap_packet(void *tapdata, packet_info *pinfo, epan_dissect_t *edt switch ( lua_pcall(tap->L,3,1,1) ) { case 0: - retval = (int)luaL_optinteger(tap->L,-1,1); + retval = luaL_optinteger(tap->L,-1,1) == 0 ? FALSE : TRUE; break; case LUA_ERRRUN: break; |