From e749545a3e4cb7664c0348067c49aa0d89caf70c Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Wed, 15 Aug 2001 06:10:42 +0000 Subject: 2001-08-12 Neal H Walfield * set.c: Do not include . Include , and . (store_set_runs): Use memcpy, not bcopy. (store_set_name): Use strdup, not a strlen, malloc and strcpy. * zero.c (zero_read): When checking if mmap failed, compare against MAP_FAILED, not -1. Use memset, not bzero. --- libstore/set.c | 13 +++++++------ libstore/zero.c | 4 ++-- 2 files changed, 9 insertions(+), 8 deletions(-) (limited to 'libstore') diff --git a/libstore/set.c b/libstore/set.c index 26a26075..a1682539 100644 --- a/libstore/set.c +++ b/libstore/set.c @@ -1,7 +1,7 @@ /* Setting various store fields - Copyright (C) 1995, 1996, 1997 Free Software Foundation, Inc. - Written by Miles Bader + Copyright (C) 1995, 1996, 1997, 2001 Free Software Foundation, Inc. + Written by Miles Bader This file is part of the GNU Hurd. The GNU Hurd is free software; you can redistribute it and/or @@ -18,8 +18,10 @@ along with this program; if not, write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111, USA. */ -#include +#include #include +#include +#include #include "store.h" @@ -37,7 +39,7 @@ store_set_runs (struct store *store, if (store->runs) free (store->runs); - bcopy (runs, copy, size); + memcpy (copy, runs, size); store->runs = copy; store->num_runs = num_runs; @@ -51,7 +53,7 @@ store_set_runs (struct store *store, error_t store_set_name (struct store *store, const char *name) { - char *copy = malloc (strlen (name) + 1); + char *copy = strdup (name); if (!copy) return ENOMEM; @@ -59,7 +61,6 @@ store_set_name (struct store *store, const char *name) if (store->name) free (store->name); - strcpy (copy, name); store->name = copy; return 0; diff --git a/libstore/zero.c b/libstore/zero.c index 3a17c1d8..b69753b3 100644 --- a/libstore/zero.c +++ b/libstore/zero.c @@ -35,13 +35,13 @@ zero_read (struct store *store, if (*len < amount) { *buf = mmap (0, amount, PROT_READ|PROT_WRITE, MAP_ANON, 0, 0); - if (*buf == (void *) -1) + if (*buf == MAP_FAILED) return errno; *len = amount; return 0; } else - bzero (*buf, amount); + memset (*buf, 0, amount); *len = amount; return 0; -- cgit v1.2.3