kernel: Don't pass the size of the var as arg2 to sysctl_handle_int().
authorSascha Wildner <saw@online.de>
Sun, 3 Aug 2014 01:08:37 +0000 (03:08 +0200)
committerSascha Wildner <saw@online.de>
Sun, 3 Aug 2014 01:08:37 +0000 (03:08 +0200)
arg1 (second parameter) is for passing a variable and arg2 (third
parameter) is for passing a constant (in which case arg1 is NULL).

14 files changed:
sys/ddb/db_sysctl.c
sys/dev/acpica/acpi_timer.c
sys/dev/sound/pci/es137x.c
sys/dev/sound/pci/via8233.c
sys/dev/sound/pcm/channel.c
sys/dev/sound/pcm/feeder_rate.c
sys/dev/sound/pcm/sndstat.c
sys/dev/sound/pcm/sound.c
sys/dev/sound/pcm/vchan.c
sys/dev/usbmisc/ubt/ubt.c
sys/dev/usbmisc/uplcom/uplcom.c
sys/dev/usbmisc/uticom/uticom.c
sys/netgraph/pppoe/ng_pppoe.c
sys/platform/pc32/i386/mp_clock.c

index 4d548f1..afd5112 100644 (file)
@@ -37,7 +37,6 @@
  * Author: Archie Cobbs <archie@whistle.com>
  *
  * $FreeBSD: src/sys/ddb/db_sysctl.c,v 1.1.4.1 2000/08/03 00:09:27 ps Exp $
- * $DragonFly: src/sys/ddb/db_sysctl.c,v 1.2 2003/06/17 04:28:20 dillon Exp $
  */
 
 #include <sys/param.h>
@@ -87,7 +86,7 @@ sysctl_debug_panic(SYSCTL_HANDLER_ARGS)
        int err;
        int val = 0;
 
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (val == 1)
                panic("sysctl_debug_panic");
        return err;
@@ -116,7 +115,7 @@ sysctl_debug_panic2(SYSCTL_HANDLER_ARGS)
        int err;
        int val = 0;
 
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (val == 1)
                stack_guard_panic2();
        return err;
index 628a9ff..d316114 100644 (file)
@@ -344,7 +344,7 @@ acpi_timer_sysctl_freq(SYSCTL_HANDLER_ARGS)
     if (acpi_cputimer.freq == 0)
        return (EOPNOTSUPP);
     freq = acpi_cputimer.freq;
-    error = sysctl_handle_int(oidp, &freq, sizeof(freq), req);
+    error = sysctl_handle_int(oidp, &freq, 0, req);
     if (error == 0 && req->newptr != NULL)
        cputimer_set_frequency(&acpi_cputimer, freq);
 
index 2e82990..34515f3 100644 (file)
@@ -1167,7 +1167,7 @@ sysctl_es137x_spdif_enable(SYSCTL_HANDLER_ARGS)
        r = es_rd(es, ES1370_REG_STATUS, 4);
        ES_UNLOCK(es);
        new_en = (r & ENABLE_SPDIF) ? 1 : 0;
-       err = sysctl_handle_int(oidp, &new_en, sizeof(new_en), req);
+       err = sysctl_handle_int(oidp, &new_en, 0, req);
 
        if (err || req->newptr == NULL)
                return (err);
@@ -1205,7 +1205,7 @@ sysctl_es137x_latency_timer(SYSCTL_HANDLER_ARGS)
        ES_LOCK(es);
        val = pci_read_config(dev, PCIR_LATTIMER, 1);
        ES_UNLOCK(es);
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        
        if (err || req->newptr == NULL)
                return (err);
@@ -1234,7 +1234,7 @@ sysctl_es137x_fixed_rate(SYSCTL_HANDLER_ARGS)
        if (val < es_caps.minspeed)
                val = 0;
        ES_UNLOCK(es);
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        
        if (err || req->newptr == NULL)
                return (err);
@@ -1289,7 +1289,7 @@ sysctl_es137x_single_pcm_mixer(SYSCTL_HANDLER_ARGS)
        set = ES_SINGLE_PCM_MIX(es->escfg);
        val = set;
        ES_UNLOCK(es);
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        
        if (err || req->newptr == NULL)
                return (err);
index 04b900e..599c53a 100644 (file)
@@ -139,7 +139,7 @@ sysctl_via8233_spdif_enable(SYSCTL_HANDLER_ARGS)
        r = pci_read_config(dev, VIA_PCI_SPDIF, 1);
        snd_mtxunlock(via->lock);
        new_en = (r & VIA_SPDIF_EN) ? 1 : 0;
-       err = sysctl_handle_int(oidp, &new_en, sizeof(new_en), req);
+       err = sysctl_handle_int(oidp, &new_en, 0, req);
 
        if (err || req->newptr == NULL)
                return err;
@@ -170,7 +170,7 @@ sysctl_via8233_dxs_src(SYSCTL_HANDLER_ARGS)
        snd_mtxlock(via->lock);
        val = via->dxs_src;
        snd_mtxunlock(via->lock);
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
 
        if (err || req->newptr == NULL)
                return err;
index 164e709..712abbe 100644 (file)
@@ -55,7 +55,7 @@ sysctl_hw_snd_targetirqrate(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = chn_targetirqrate;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (val < 16 || val > 512)
                err = EINVAL;
        else
index 4365eb5..5eaade6 100644 (file)
@@ -140,7 +140,7 @@ sysctl_hw_snd_feeder_rate_ratemin(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = feeder_rate_ratemin;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (val < 1 || val >= feeder_rate_ratemax)
                err = EINVAL;
        else
@@ -156,7 +156,7 @@ sysctl_hw_snd_feeder_rate_ratemax(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = feeder_rate_ratemax;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (val <= feeder_rate_ratemin || val > 0x7fffff)
                err = EINVAL;
        else
@@ -172,7 +172,7 @@ sysctl_hw_snd_feeder_rate_scaling(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = feeder_rate_scaling;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        /*
         *      Feeder Scaling Type
         *      ===================
@@ -215,7 +215,7 @@ sysctl_hw_snd_feeder_rate_buffersize(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = feeder_rate_buffersize;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        /*
         * Don't waste too much kernel space
         */
index 59f6a5f..f890104 100644 (file)
@@ -86,7 +86,7 @@ sysctl_hw_sndverbose(SYSCTL_HANDLER_ARGS)
        int error, verbose;
 
        verbose = sndstat_verbose;
-       error = sysctl_handle_int(oidp, &verbose, sizeof(verbose), req);
+       error = sysctl_handle_int(oidp, &verbose, 0, req);
        if (error == 0 && req->newptr != NULL) {
                lockmgr(&sndstat_lock, LK_EXCLUSIVE);
                if (verbose < 0 || verbose > 3)
index 8f2a455..4076d5f 100644 (file)
@@ -25,7 +25,6 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/sound/pcm/sound.c,v 1.93.2.5 2007/06/04 09:06:05 ariff Exp $
- * $DragonFly: src/sys/dev/sound/pcm/sound.c,v 1.12 2008/01/06 16:55:51 swildner Exp $
  */
 
 #include <dev/sound/pcm/sound.h>
@@ -362,7 +361,7 @@ sysctl_hw_snd_unit(SYSCTL_HANDLER_ARGS)
        int error, unit;
 
        unit = snd_unit;
-       error = sysctl_handle_int(oidp, &unit, sizeof(unit), req);
+       error = sysctl_handle_int(oidp, &unit, 0, req);
        if (error == 0 && req->newptr != NULL) {
                if (unit < 0 || (pcm_devclass != NULL &&
                    unit >= devclass_get_maxunit(pcm_devclass)))
@@ -385,7 +384,7 @@ sysctl_hw_snd_maxautovchans(SYSCTL_HANDLER_ARGS)
        int i, v, error;
 
        v = snd_maxautovchans;
-       error = sysctl_handle_int(oidp, &v, sizeof(v), req);
+       error = sysctl_handle_int(oidp, &v, 0, req);
        if (error == 0 && req->newptr != NULL) {
                if (v < 0 || v > PCMMAXCHAN)
                        return E2BIG;
@@ -1087,7 +1086,7 @@ sysctl_hw_snd_vchans(SYSCTL_HANDLER_ARGS)
        d = oidp->oid_arg1;
 
        newcnt = d->vchancount;
-       err = sysctl_handle_int(oidp, &newcnt, sizeof(newcnt), req);
+       err = sysctl_handle_int(oidp, &newcnt, 0, req);
 
        if (err == 0 && req->newptr != NULL && d->vchancount != newcnt)
                err = pcm_setvchans(d, newcnt);
index e3b029f..7dcaf24 100644 (file)
@@ -356,7 +356,7 @@ sysctl_hw_snd_vchanrate(SYSCTL_HANDLER_ARGS)
                pcm_inprog(d, -1);
                return EINVAL;
        }
-       err = sysctl_handle_int(oidp, &newspd, sizeof(newspd), req);
+       err = sysctl_handle_int(oidp, &newspd, 0, req);
        if (err == 0 && req->newptr != NULL) {
                if (newspd < 1 || newspd < feeder_rate_ratemin ||
                                newspd > feeder_rate_ratemax) {
index 11fdc97..e336bab 100644 (file)
@@ -651,7 +651,7 @@ ubt_sysctl_config(SYSCTL_HANDLER_ARGS)
        int t, error;
 
        t = sc->sc_config;
-       error = sysctl_handle_int(oidp, &t, sizeof(t), req);
+       error = sysctl_handle_int(oidp, &t, 0, req);
        if (error || req->newptr == NULL)
                return error;
 
index 2ba1231..3fb4284 100644 (file)
@@ -283,7 +283,7 @@ sysctl_hw_usb_uplcom_interval(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = uplcominterval;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (err != 0 || req->newptr == NULL)
                return (err);
        if (0 < val && val <= 1000)
index 6d9c88c..adfa843 100644 (file)
@@ -211,7 +211,7 @@ sysctl_hw_usb_uticom_stickdsr(SYSCTL_HANDLER_ARGS)
        int err, val;
 
        val = uticomstickdsr;
-       err = sysctl_handle_int(oidp, &val, sizeof(val), req);
+       err = sysctl_handle_int(oidp, &val, 0, req);
        if (err != 0 || req->newptr == NULL)
                return (err);
        if (val == 0 || val == 1)
index d3ed66f..af9b869 100644 (file)
@@ -181,7 +181,7 @@ ngpppoe_set_ethertype(SYSCTL_HANDLER_ARGS)
        int val;
 
        val = pppoe_mode;
-       error = sysctl_handle_int(oidp, &val, sizeof(int), req);
+       error = sysctl_handle_int(oidp, &val, 0, req);
        if (error != 0 || req->newptr == NULL)
                return (error);
        switch (val) {
index 7425ae6..44249e5 100644 (file)
@@ -68,7 +68,7 @@ sysctl_machdep_piix_freq(SYSCTL_HANDLER_ARGS)
        if (piix_timecounter.tc_frequency == 0)
                return (EOPNOTSUPP);
        freq = piix_freq;
-       error = sysctl_handle_int(oidp, &freq, sizeof(freq), req);
+       error = sysctl_handle_int(oidp, &freq, 0, req);
        if (error == 0 && req->newptr != NULL) {
                piix_freq = freq;
                piix_timecounter.tc_frequency = piix_freq;