----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38770/#review100661 -----------------------------------------------------------
Ship it! Or you can do unconditionally os::close() after that ioctl() to save a few code... but this is very minor. - Cong Wang On Sept. 25, 2015, 7:07 p.m., Chi Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38770/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2015, 7:07 p.m.) > > > Review request for mesos, Ben Mahler, Jie Yu, and Cong Wang. > > > Repository: mesos > > > Description > ------- > > routing: fixed a file descriptor leak. > > > Diffs > ----- > > src/linux/routing/link/link.cpp 552e2d76a6f7d2ffc366a65957d0ac849b504b2e > > Diff: https://reviews.apache.org/r/38770/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Chi Zhang > >
