Hi Ryan, Thanks for the comments. I think free() in libc internally will check if remotes_error is NULL or not. Regards, William > > } > > > > + free(remotes_error); > > Shouldn't this be > > if (remotes_error) { > free(remotes_error); > } > > Ryan Moats >