diff options
| author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2000-08-24 08:55:30 +0000 |
|---|---|---|
| committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2000-08-24 08:55:30 +0000 |
| commit | 69c50e7ddb549fa82a5dd5ecd58b27c393171ecb (patch) | |
| tree | a0439711985bb83eb8b4580d3788e58adeb8e25d /packet-rpc.c | |
| parent | a44fe9e67490d723c619dd55f1daae7a3a746425 (diff) | |
Use 0, rather that 0xffffffff, as the "no reply frame number known yet"
- frame numbers are 1-origin, so 0 can be used as an "exception" value.
In the protocol tree for a reply, don't say that the reply is to frame
N, just say that it's to a request starting in frame N - a frame can
contain more than one request, and a request may take more than one
frame.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2360 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'packet-rpc.c')
| -rw-r--r-- | packet-rpc.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/packet-rpc.c b/packet-rpc.c index 84727505c4..c0fa0405bb 100644 --- a/packet-rpc.c +++ b/packet-rpc.c @@ -2,7 +2,7 @@ * Routines for rpc dissection * Copyright 1999, Uwe Girlich <Uwe.Girlich@philosys.de> * - * $Id: packet-rpc.c,v 1.38 2000/08/24 06:19:52 guy Exp $ + * $Id: packet-rpc.c,v 1.39 2000/08/24 08:55:30 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -1288,9 +1288,13 @@ dissect_rpc( const u_char *pd, int offset, frame_data *fd, proto_tree *tree) } } else { - /* prepare the value data */ + /* Prepare the value data. + "req_num" and "rep_num" are frame numbers; + frame numbers are 1-origin, so we use 0 + to mean "we don't yet know in which frame + the reply for this call appears". */ rpc_call_msg.req_num = fd->num; - rpc_call_msg.rep_num = 0xffffffff; + rpc_call_msg.rep_num = 0; rpc_call_msg.prog = prog; rpc_call_msg.vers = vers; rpc_call_msg.proc = proc; @@ -1323,7 +1327,7 @@ dissect_rpc( const u_char *pd, int offset, frame_data *fd, proto_tree *tree) /* Indicate the frame to which this is a reply. */ proto_tree_add_text(rpc_tree, NullTVB, 0, 0, - "This is a reply to frame %u", + "This is a reply to a request starting in frame %u", rpc_call->req_num); if (rpc_call->proc_info != NULL) { @@ -1380,7 +1384,7 @@ dissect_rpc( const u_char *pd, int offset, frame_data *fd, proto_tree *tree) "Procedure: %s (%u)", procname, proc); } - if (rpc_call->rep_num == 0xffffffff) { + if (rpc_call->rep_num == 0) { /* We have not yet seen a reply to that call, so this must be the first reply; remember its frame number. */ |