diff options
author | Guy Harris <guy@alum.mit.edu> | 2011-03-22 07:42:45 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2011-03-22 07:42:45 +0000 |
commit | 423e428441afd338600fc21464454ea6c8c22f52 (patch) | |
tree | d96dcab34dc36bd9346a8d8f7a6ead1113e7163d /wiretap/netscreen.c | |
parent | 3a39cb2ed675bd8b22cf2291383fac36125c8ad0 (diff) |
Use the same style in netscreen_seek_read() that we use in
netscreen_read(), checking the return value of
parse_netscreen_hex_dump() against -1 and explicitly returning FALSE if
it's -1, otherwise driving on.
svn path=/trunk/; revision=36237
Diffstat (limited to 'wiretap/netscreen.c')
-rw-r--r-- | wiretap/netscreen.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/wiretap/netscreen.c b/wiretap/netscreen.c index 037aba1619..afdb1adab3 100644 --- a/wiretap/netscreen.c +++ b/wiretap/netscreen.c @@ -322,7 +322,11 @@ netscreen_seek_read (wtap *wth, gint64 seek_off, return FALSE; } - return (parse_netscreen_hex_dump(wth->random_fh, len, pd, err, err_info) != -1); + if (parse_netscreen_hex_dump(wth->random_fh, len, pd, err, err_info) + == -1) { + return FALSE; + } + return TRUE; } /* Parses a packet record header. There are a few possible formats: |