>-----Original Message----- >From: Ben Pfaff [mailto:b...@ovn.org] >Sent: Monday, August 8, 2016 5:24 PM >To: Bodireddy, Bhanuprakash <bhanuprakash.bodire...@intel.com> >Cc: nickcooper-zhangtonghao <nickcooper-zhangtong...@opencloud.tech>; >dev@openvswitch.org >Subject: Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Fix potential crash with >timeout argument > >On Mon, Aug 08, 2016 at 03:29:11PM +0000, Bodireddy, Bhanuprakash wrote: >> >It seems to me that it’s unnecessary to change the codes. If the >> >“timeout” is empty, the “atoi” function will convert it to 0, >> >> Got a signal when I passed empty timeout accidentally during my test runs >and so I sent out this patch. Don’t you see this issue on your target? >> # ovs-appctl --timeout= dpif-netdev/pmd-stats-show >> 2016-08-08T15:20:01Z|00001|fatal_signal|WARN|terminating with signal >> 14 (Alarm clock) Alarm clock > >The signal is the timeout expiring! SIGALRM seems like a reasonable way to >terminate a program that is timing out. Not a bug.
Thanks Ben for the clarification. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
