Hi tech@,

Add sizes for free() for octeon.

Comments? OK?

Index: sys/arch/octeon/dev/amdcf.c
===================================================================
RCS file: /cvs/src/sys/arch/octeon/dev/amdcf.c,v
retrieving revision 1.5
diff -u -p -r1.5 amdcf.c
--- sys/arch/octeon/dev/amdcf.c 30 Dec 2017 23:08:29 -0000      1.5
+++ sys/arch/octeon/dev/amdcf.c 30 Apr 2018 21:40:24 -0000
@@ -495,7 +495,7 @@ amdcfioctl(dev_t dev, u_long xfer, caddr
                lp = malloc(sizeof(*lp), M_TEMP, M_WAITOK);
                amdcfgetdisklabel(dev, sc, lp, 0);
                bcopy(lp, sc->sc_dk.dk_label, sizeof(*lp));
-               free(lp, M_TEMP, 0);
+               free(lp, M_TEMP, sizeof(*lp));
                goto exit;
 
        case DIOCGPDINFO:
Index: sys/arch/octeon/dev/octcf.c
===================================================================
RCS file: /cvs/src/sys/arch/octeon/dev/octcf.c,v
retrieving revision 1.31
diff -u -p -r1.31 octcf.c
--- sys/arch/octeon/dev/octcf.c 30 Dec 2017 23:08:29 -0000      1.31
+++ sys/arch/octeon/dev/octcf.c 30 Apr 2018 21:40:24 -0000
@@ -563,7 +563,7 @@ octcfioctl(dev_t dev, u_long xfer, caddr
                lp = malloc(sizeof(*lp), M_TEMP, M_WAITOK);
                octcfgetdisklabel(dev, wd, lp, 0);
                bcopy(lp, wd->sc_dk.dk_label, sizeof(*lp));
-               free(lp, M_TEMP, 0);
+               free(lp, M_TEMP, sizeof(*lp));
                goto exit;
 
        case DIOCGPDINFO:
@@ -788,7 +788,7 @@ octcf_get_params(struct octcf_softc *wd,
 
        if (error != 0) {
                printf("%s: identify failed: %d\n", __func__, error);
-               free(tb, M_DEVBUF, 0);
+               free(tb, M_DEVBUF, ATAPARAMS_SIZE);
                return CMD_ERR;
        } else {
                /*
@@ -818,7 +818,7 @@ octcf_get_params(struct octcf_softc *wd,
                        params->atap_model[1] == 'E') ||
                     (params->atap_model[0] == 'F' &&
                         params->atap_model[1] == 'X'))) {
-                       free(tb, M_DEVBUF, 0);
+                       free(tb, M_DEVBUF, ATAPARAMS_SIZE);
                        return CMD_OK;
                }
                for (i = 0; i < sizeof(params->atap_model); i += 2) {
@@ -834,7 +834,7 @@ octcf_get_params(struct octcf_softc *wd,
                        *p = swap16(*p);
                }
 
-               free(tb, M_DEVBUF, 0);
+               free(tb, M_DEVBUF, ATAPARAMS_SIZE);
                return CMD_OK;
        }
 }

Reply via email to