On Sun, 10 Feb 2013, Danny Al-Gaaf wrote: > Am 10.02.2013 07:04, schrieb Sage Weil: > > On Fri, 8 Feb 2013, Danny Al-Gaaf wrote: > [...] > >> //dout(5) << "SyntheticClient::write_fd: writing to fd " << fd << dendl; > >> - if (fd < 0) return fd; > >> + if (fd < 0) { > >> + return fd; > >> + delete[] buf; > > > > oops, wrong order... > > I know, that's why I've send another fixed version of this patch to the > list the same day: > http://thread.gmane.org/gmane.comp.file-systems.ceph.devel/12969/focus=12985 Oops, I see it now, sorry :) I merged the memleaks branch. The cppcheck-clang one is coming up with build errors tho: http://ceph.com/gitbuilder.cgi Thanks! sage -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html