Fix more wrong sizeof() usages, part 3/x
authorSascha Wildner <saw@online.de>
Thu, 21 Jun 2012 02:25:19 +0000 (04:25 +0200)
committerSascha Wildner <saw@online.de>
Thu, 21 Jun 2012 02:40:17 +0000 (04:40 +0200)
Some comments:

* The gzip fix speeds up uncompression of compress(1)'d files with gzip.
  A test here took 9s with the fix when it previously took 15s.

* Our OpenSSH is without J-PAKE support, hence the OpenSSH fix is only
  cosmetical.

Found-with: Coccinelle (http://coccinelle.lip6.fr/)

crypto/openssh/jpake.c
sys/dev/acpica5/acpi_package.c
sys/netproto/ncp/ncp_mod.c
usr.bin/gzip/zuncompress.c

index ac9a4bc..c5a7da4 100644 (file)
@@ -133,7 +133,7 @@ jpake_free(struct jpake_ctx *pctx)
 #undef JPAKE_BN_CLEAR_FREE
 #undef JPAKE_BUF_CLEAR_FREE
 
-       bzero(pctx, sizeof(pctx));
+       bzero(pctx, sizeof(*pctx));
        xfree(pctx);
 }
 
index 31e3e22..e8a367e 100644 (file)
@@ -76,7 +76,7 @@ acpi_PkgStr(ACPI_OBJECT *res, int idx, void *dst, size_t size)
     obj = &res->Package.Elements[idx];
     if (obj == NULL)
        return (EINVAL);
-    bzero(dst, sizeof(dst));
+    bzero(dst, size);
 
     switch (obj->Type) {
     case ACPI_TYPE_STRING:
index 22c99ba..1b91a82 100644 (file)
@@ -100,7 +100,8 @@ sys_sncp_connect(struct sncp_connect_args *uap)
        if (!error) {
                error = ncp_conn_gethandle(conn, td, &handle);
                if (error == 0)
-                       copyout(&handle->nh_id, uap->connHandle, sizeof(uap->connHandle));
+                       copyout(&handle->nh_id, uap->connHandle,
+                           sizeof(*uap->connHandle));
                ncp_conn_unlock(conn,td);
        }
        rel_mplock();
index 34bc8b2..db5d866 100644 (file)
@@ -1,5 +1,4 @@
 /*     $NetBSD: zuncompress.c,v 1.11 2011/08/16 13:55:02 joerg Exp $ */
-/*     $DragonFly: src/usr.bin/gzip/zuncompress.c,v 1.1 2004/10/26 11:19:31 joerg Exp $ */
 
 /*-
  * Copyright (c) 1985, 1986, 1992, 1993
@@ -146,7 +145,7 @@ zuncompress(FILE *in, FILE *out, char *pre, size_t prelen,
        else
                compressed_pre = NULL;
 
-       while ((bin = fread(buf, 1, sizeof(buf), in)) != 0) {
+       while ((bin = fread(buf, 1, BUFSIZE, in)) != 0) {
                if (tflag == 0 && (off_t)fwrite(buf, 1, bin, out) != bin) {
                        free(buf);
                        return -1;