diff options
| author | Ben Pfaff <blp@ovn.org> | 2017-05-26 16:03:54 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2017-06-01 16:20:45 -0700 |
| commit | 63ae068ebf3d0ff7e3cb8d810ef87766187b7821 (patch) | |
| tree | 5ea71c356dfb5a817f297b114c094c1aa2c17109 /utilities/ovs-ofctl.c | |
| parent | 08125baff54544d54e66155bc6274176bf96ee92 (diff) | |
| download | openvswitch-63ae068ebf3d0ff7e3cb8d810ef87766187b7821.tar.gz | |
ovs-ofctl: Close pcap file at end of ofctl_ofp_parse_pcap().
Seem wise even if not mandatory.
Found by Coverity.
Reported-at: https://scan3.coverity.com/reports.htm#v16889/p10449/fileInstanceId=14763047&defectInstanceId=4305314&mergedDefectId=180420
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Justin Pettit <jpettit@ovn.org>
Diffstat (limited to 'utilities/ovs-ofctl.c')
| -rw-r--r-- | utilities/ovs-ofctl.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c index 90043bf41..dca9be3a5 100644 --- a/utilities/ovs-ofctl.c +++ b/utilities/ovs-ofctl.c @@ -2591,6 +2591,7 @@ ofctl_ofp_parse_pcap(struct ovs_cmdl_context *ctx) dp_packet_delete(packet); } tcp_reader_close(reader); + fclose(file); } static void |