Hi Anchit, On Thu, Jul 30, 2015, Anchit Narang wrote: > Removed dynamic allocation of filename and savefile by strdup. > Instead filename and savefile are directly pointing to the argv element. > Closed the file before returning from do_send function. > --- > tools/rctest.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) I know I let it slip last time, but in principle independent fixes should be in separate patches. So please split the fd leak to a separate patch. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html