Re: small mv patch

2015-08-08 Thread Philip Guenther
On Fri, Aug 7, 2015 at 12:24 AM, Martijn van Duren wrote: > Hello tech@, > > I was reading mv.c and found two minor things in fastcopy: > 1) fd leak on seldom reached code I think this one is handled more cleanly by moving the "if (!blen)" block up before two open()s. That way if the malloc fail

small mv patch

2015-08-07 Thread Martijn van Duren
Hello tech@, I was reading mv.c and found two minor things in fastcopy: 1) fd leak on seldom reached code 2) At the end of the function utimes is called, while the rest of the code calls the f* variant. Consistency fix. Index: mv.c ==