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 <b...@ovn.org> --- utilities/ovs-ofctl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c index 1a5e2345b7d4..140cd18e617a 100644 --- a/utilities/ovs-ofctl.c +++ b/utilities/ovs-ofctl.c @@ -2484,6 +2484,7 @@ ofctl_ofp_parse_pcap(struct ovs_cmdl_context *ctx) dp_packet_delete(packet); } tcp_reader_close(reader); + fclose(file); } static void -- 2.10.2 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev