=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/ssh/sftp-client.c,v retrieving revision 1.97.4.1 retrieving revision 1.98 diff -u -r1.97.4.1 -r1.98 --- src/usr.bin/ssh/sftp-client.c 2013/11/08 05:52:21 1.97.4.1 +++ src/usr.bin/ssh/sftp-client.c 2013/05/17 00:13:14 1.98 @@ -1,4 +1,4 @@ -/* $OpenBSD: sftp-client.c,v 1.97.4.1 2013/11/08 05:52:21 djm Exp $ */ +/* $OpenBSD: sftp-client.c,v 1.98 2013/05/17 00:13:14 djm Exp $ */ /* * Copyright (c) 2001-2004 Damien Miller * @@ -387,8 +387,8 @@ } else { debug2("Unrecognised server extension \"%s\"", name); } - xfree(name); - xfree(value); + free(name); + free(value); } buffer_free(&msg); @@ -464,7 +464,7 @@ if (dir) { ents = 0; - *dir = xcalloc(1, sizeof(**dir)); + *dir = xmalloc(sizeof(**dir)); (*dir)[0] = NULL; } @@ -502,7 +502,7 @@ error("Couldn't read directory: %s", fx2txt(status)); do_close(conn, handle, handle_len); - xfree(handle); + free(handle); buffer_free(&msg); return(status); } @@ -538,26 +538,26 @@ if (dir) { *dir = xrealloc(*dir, ents + 2, sizeof(**dir)); - (*dir)[ents] = xcalloc(1, sizeof(***dir)); + (*dir)[ents] = xmalloc(sizeof(***dir)); (*dir)[ents]->filename = xstrdup(filename); (*dir)[ents]->longname = xstrdup(longname); memcpy(&(*dir)[ents]->a, a, sizeof(*a)); (*dir)[++ents] = NULL; } next: - xfree(filename); - xfree(longname); + free(filename); + free(longname); } } buffer_free(&msg); do_close(conn, handle, handle_len); - xfree(handle); + free(handle); /* Don't return partial matches on interrupt */ if (interrupted && dir != NULL && *dir != NULL) { free_sftp_dirents(*dir); - *dir = xcalloc(1, sizeof(**dir)); + *dir = xmalloc(sizeof(**dir)); **dir = NULL; } @@ -575,11 +575,11 @@ int i; for (i = 0; s[i]; i++) { - xfree(s[i]->filename); - xfree(s[i]->longname); - xfree(s[i]); + free(s[i]->filename); + free(s[i]->longname); + free(s[i]); } - xfree(s); + free(s); } int @@ -753,7 +753,7 @@ debug3("SSH_FXP_REALPATH %s -> %s size %lu", path, filename, (unsigned long)a->size); - xfree(longname); + free(longname); buffer_free(&msg); @@ -900,7 +900,7 @@ debug3("SSH_FXP_READLINK %s -> %s", path, filename); - xfree(longname); + free(longname); buffer_free(&msg); @@ -1050,7 +1050,7 @@ local_path, strerror(errno)); do_close(conn, handle, handle_len); buffer_free(&msg); - xfree(handle); + free(handle); return(-1); } @@ -1082,7 +1082,7 @@ (unsigned long long)offset, (unsigned long long)offset + buflen - 1, num_req, max_req); - req = xcalloc(1, sizeof(*req)); + req = xmalloc(sizeof(*req)); req->id = conn->msg_id++; req->len = buflen; req->offset = offset; @@ -1114,7 +1114,7 @@ read_error = 1; max_req = 0; TAILQ_REMOVE(&requests, req, tq); - xfree(req); + free(req); num_req--; break; case SSH2_FXP_DATA: @@ -1133,11 +1133,11 @@ max_req = 0; } progress_counter += len; - xfree(data); + free(data); if (len == req->len) { TAILQ_REMOVE(&requests, req, tq); - xfree(req); + free(req); num_req--; } else { /* Resend the request for the missing data */ @@ -1209,7 +1209,7 @@ } close(local_fd); buffer_free(&msg); - xfree(handle); + free(handle); return(status); } @@ -1281,8 +1281,8 @@ } else logit("%s: not a regular file\n", new_src); - xfree(new_dst); - xfree(new_src); + free(new_dst); + free(new_src); } if (pflag) { @@ -1318,7 +1318,7 @@ ret = download_dir_internal(conn, src_canon, dst, dirattrib, pflag, printflag, 0); - xfree(src_canon); + free(src_canon); return ret; } @@ -1421,7 +1421,7 @@ strerror(errno)); if (len != 0) { - ack = xcalloc(1, sizeof(*ack)); + ack = xmalloc(sizeof(*ack)); ack->id = ++id; ack->offset = offset; ack->len = len; @@ -1469,7 +1469,7 @@ debug3("In write loop, ack for %u %u bytes at %lld", ack->id, ack->len, (long long)ack->offset); ++ackid; - xfree(ack); + free(ack); } offset += len; if (offset < 0) @@ -1479,7 +1479,7 @@ if (showprogress) stop_progress_meter(); - xfree(data); + free(data); if (status != SSH2_FX_OK) { error("Couldn't write to remote file \"%s\": %s", @@ -1499,7 +1499,7 @@ if (do_close(conn, handle, handle_len) != SSH2_FX_OK) status = -1; - xfree(handle); + free(handle); return status; } @@ -1585,8 +1585,8 @@ } } else logit("%s: not a regular file\n", filename); - xfree(new_dst); - xfree(new_src); + free(new_dst); + free(new_src); } do_setstat(conn, dst, &a); @@ -1608,7 +1608,7 @@ } ret = upload_dir_internal(conn, src, dst_canon, pflag, printflag, 0); - xfree(dst_canon); + free(dst_canon); return ret; }