diff options
| author | Luis Ontanon <luis.ontanon@gmail.com> | 2006-03-08 10:14:51 +0000 |
|---|---|---|
| committer | Luis Ontanon <luis.ontanon@gmail.com> | 2006-03-08 10:14:51 +0000 |
| commit | 3fa91ab7be5ad84471eda9c6a4e75e748412aaf4 (patch) | |
| tree | 1f9a3f9609b8d94a0f1a89580a17efe2707ac0ba /wiretap/network_instruments.c | |
| parent | 8284947d8432106dbe934121671454555f93a50d (diff) | |
| download | wireshark-3fa91ab7be5ad84471eda9c6a4e75e748412aaf4.tar.gz wireshark-3fa91ab7be5ad84471eda9c6a4e75e748412aaf4.tar.bz2 wireshark-3fa91ab7be5ad84471eda9c6a4e75e748412aaf4.zip | |
Coverity's run 14 CID 82.
an off by one error (> vs >= in bounds check).
svn path=/trunk/; revision=17520
Diffstat (limited to 'wiretap/network_instruments.c')
| -rw-r--r-- | wiretap/network_instruments.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/wiretap/network_instruments.c b/wiretap/network_instruments.c index 7d723c4804..7a3ac43b36 100644 --- a/wiretap/network_instruments.c +++ b/wiretap/network_instruments.c @@ -354,8 +354,8 @@ int network_instruments_dump_can_write_encap(int encap) /* Per-packet encapsulations aren't supported. */ if (encap == WTAP_ENCAP_PER_PACKET) return WTAP_ERR_ENCAP_PER_PACKET_UNSUPPORTED; - - if (encap < 0 || (unsigned) encap > NUM_FROM_WTAP_ENCAPS || from_wtap_encap[encap] == OBSERVER_UNDEFINED) + + if (encap < 0 || (unsigned) encap >= NUM_FROM_WTAP_ENCAPS || from_wtap_encap[encap] == OBSERVER_UNDEFINED) return WTAP_ERR_UNSUPPORTED_ENCAP; return 0; |