drivers/efuse/efuse: Multiple Drivers Are Registered With World Writable

arch/arm/src/c5471/c5471_watchdog.c

@@ -359,7 +359,7 @@ int arm_wdtinit(void)
 
   /* Register as /dev/wdt */
 
-  ret = register_driver("/dev/wdt", &g_wdtops, 0666, NULL);
+  ret = register_driver("/dev/wdt", &g_wdtops, 0600, NULL);
   if (ret)
     {
       return ERROR;

arch/arm/src/common/stm32/stm32_bbsram_m3m4_v1.c

@@ -757,7 +757,7 @@ int stm32_bbsraminitialize(char *devpath, int *sizes)
     {
       snprintf(devname, sizeof(devname), "%s%d", devpath, i);
       ret = register_driver(devname, &g_stm32_bbsram_fops,
-                            0666, &g_bbsram[i]);
+                            0600, &g_bbsram[i]);
     }
 
   /* Disallow Access */

arch/arm/src/common/stm32/stm32_hrtim_m3m4_v1.c

@@ -6037,7 +6037,7 @@ int hrtim_register(const char *path, struct hrtim_dev_s *dev)
 
   /* Register the HRTIM character driver */
 
-  return register_driver(path, &g_hrtim_fops, 0444, dev);
+  return register_driver(path, &g_hrtim_fops, 0440, dev);
 }
 #endif /* CONFIG_STM32_HRTIM_DISABLE_CHARDRV */
 

arch/arm/src/cxd56xx/cxd56_adc.c

@@ -1105,7 +1105,7 @@ int cxd56_adcinitialize(void)
   int ret = OK;
 
 #if defined (CONFIG_CXD56_LPADC0) || defined (CONFIG_CXD56_LPADC0_1) || defined (CONFIG_CXD56_LPADC_ALL)
-  ret = register_driver("/dev/lpadc0", &g_adcops, 0666, &g_lpadc0priv);
+  ret = register_driver("/dev/lpadc0", &g_adcops, 0600, &g_lpadc0priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(lpadc0): %d\n", ret);
@@ -1114,7 +1114,7 @@ int cxd56_adcinitialize(void)
 #endif
 
 #if defined (CONFIG_CXD56_LPADC1) || defined (CONFIG_CXD56_LPADC0_1) || defined (CONFIG_CXD56_LPADC_ALL)
-  ret = register_driver("/dev/lpadc1", &g_adcops, 0666, &g_lpadc1priv);
+  ret = register_driver("/dev/lpadc1", &g_adcops, 0600, &g_lpadc1priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(lpadc1): %d\n", ret);
@@ -1123,7 +1123,7 @@ int cxd56_adcinitialize(void)
 #endif
 
 #if defined (CONFIG_CXD56_LPADC2) || defined (CONFIG_CXD56_LPADC_ALL)
-  ret = register_driver("/dev/lpadc2", &g_adcops, 0666, &g_lpadc2priv);
+  ret = register_driver("/dev/lpadc2", &g_adcops, 0600, &g_lpadc2priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(lpadc2): %d\n", ret);
@@ -1132,7 +1132,7 @@ int cxd56_adcinitialize(void)
 #endif
 
 #if defined (CONFIG_CXD56_LPADC3) || defined (CONFIG_CXD56_LPADC_ALL)
-  ret = register_driver("/dev/lpadc3", &g_adcops, 0666, &g_lpadc3priv);
+  ret = register_driver("/dev/lpadc3", &g_adcops, 0600, &g_lpadc3priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(lpadc3): %d\n", ret);
@@ -1141,7 +1141,7 @@ int cxd56_adcinitialize(void)
 #endif
 
 #ifdef CONFIG_CXD56_HPADC0
-  ret = register_driver("/dev/hpadc0", &g_adcops, 0666, &g_hpadc0priv);
+  ret = register_driver("/dev/hpadc0", &g_adcops, 0600, &g_hpadc0priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(hpadc0): %d\n", ret);
@@ -1150,7 +1150,7 @@ int cxd56_adcinitialize(void)
 #endif
 
 #ifdef CONFIG_CXD56_HPADC1
-  ret = register_driver("/dev/hpadc1", &g_adcops, 0666, &g_hpadc1priv);
+  ret = register_driver("/dev/hpadc1", &g_adcops, 0600, &g_hpadc1priv);
   if (ret < 0)
     {
       aerr("Failed to register driver(hpadc1): %d\n", ret);

arch/arm/src/cxd56xx/cxd56_charger.c

@@ -623,7 +623,7 @@ int cxd56_charger_initialize(const char *devpath)
 
   /* Register battery driver */
 
-  ret = register_driver(devpath, &g_chargerops, 0666, priv);
+  ret = register_driver(devpath, &g_chargerops, 0600, priv);
   if (ret < 0)
     {
       baterr("ERROR: register_driver failed: %d\n", ret);

arch/arm/src/cxd56xx/cxd56_gauge.c

@@ -347,7 +347,7 @@ int cxd56_gauge_initialize(const char *devpath)
 
   /* Register battery driver */
 
-  ret = register_driver(devpath, &g_gaugeops, 0666, priv);
+  ret = register_driver(devpath, &g_gaugeops, 0600, priv);
   if (ret < 0)
     {
       baterr("ERROR: register_driver failed: %d\n", ret);

arch/arm/src/cxd56xx/cxd56_ge2d.c

@@ -186,7 +186,7 @@ int cxd56_ge2dinitialize(const char *devname)
 {
   int ret;
 
-  ret = register_driver(devname, &g_ge2dfops, 0666, NULL);
+  ret = register_driver(devname, &g_ge2dfops, 0600, NULL);
   if (ret != 0)
     {
       return ERROR;

arch/arm/src/cxd56xx/cxd56_geofence.c

@@ -649,7 +649,7 @@ static int cxd56_geofence_register(const char *devpath)
       goto err0;
     }
 
-  ret = register_driver(devpath, &g_geofencefops, 0666, priv);
+  ret = register_driver(devpath, &g_geofencefops, 0600, priv);
   if (ret < 0)
     {
       gnsserr("Failed to register driver: %d\n", ret);

arch/arm/src/cxd56xx/cxd56_gnss.c

@@ -3207,7 +3207,7 @@ static int cxd56_gnss_register(const char *devpath)
       goto err0;
     }
 
-  ret = register_driver(devpath, &g_gnssfops, 0666, priv);
+  ret = register_driver(devpath, &g_gnssfops, 0600, priv);
   if (ret < 0)
     {
       gnsserr("Failed to register driver: %d\n", ret);

arch/arm/src/cxd56xx/cxd56_hostif.c

@@ -416,7 +416,7 @@ static int hif_initialize(struct hostif_buff_s *buffer)
       snprintf(devpath, sizeof(devpath), "/dev/hostif%c%d",
                (priv->flags & HOSTIF_BUFF_ATTR_READ) ? 'r' : 'w', num);
 
-      ret = register_driver(devpath, &g_hif_fops, 0666, priv);
+      ret = register_driver(devpath, &g_hif_fops, 0600, priv);
       if (ret < 0)
         {
           hiferr("ERROR: Failed to register %s (%d)\n", devpath, ret);

arch/arm/src/cxd56xx/cxd56_sph.c

@@ -226,7 +226,7 @@ static inline int cxd56_sphdevinit(const char *devname, int num)
 
   snprintf(fullpath, sizeof(fullpath), "/dev/%s%d", devname, num);
 
-  ret = register_driver(fullpath, &g_sph_fops, 0666, (void *)priv);
+  ret = register_driver(fullpath, &g_sph_fops, 0600, (void *)priv);
   if (ret != 0)
     {
       return ERROR;

arch/arm/src/cxd56xx/cxd56_sysctl.c

@@ -162,5 +162,5 @@ void cxd56_sysctlinitialize(void)
 {
   cxd56_iccinit(CXD56_PROTO_SYSCTL);
   cxd56_iccregisterhandler(CXD56_PROTO_SYSCTL, sysctl_rxhandler, NULL);
-  register_driver("/dev/sysctl", &g_sysctlfops, 0666, NULL);
+  register_driver("/dev/sysctl", &g_sysctlfops, 0600, NULL);
 }

arch/arm/src/cxd56xx/cxd56_uart0.c

@@ -233,7 +233,7 @@ static ssize_t uart0_write(struct file *filep,
 
 int cxd56_uart0initialize(const char *devname)
 {
-  return register_driver(devname, &g_uart0fops, 0666, NULL);
+  return register_driver(devname, &g_uart0fops, 0600, NULL);
 }
 
 /****************************************************************************

arch/arm/src/phy62xx/phyplus_stub.c

@@ -752,7 +752,7 @@ int phyplus_stub_register(void)
 {
   char devname[16];
   snprintf(devname, sizeof(devname), "/dev/phyplus");
-  return register_driver(devname, &g_stub_drvrops, 0666, NULL);
+  return register_driver(devname, &g_stub_drvrops, 0600, NULL);
 }
 
 /****************************************************************************

arch/arm/src/rtl8720c/ameba_hci.c

@@ -507,6 +507,6 @@ static hci_dev_t g_hcidev =
 
 int amebaz_bt_hci_uart_register(const char *path)
 {
-  return register_driver(path, &g_hcidev.i_ops, 0666, &g_hcidev);
+  return register_driver(path, &g_hcidev.i_ops, 0600, &g_hcidev);
 }
 

arch/arm/src/sama5/sam_tsd.c

@@ -1818,7 +1818,7 @@ int sam_tsd_register(struct sam_adc_s *adc, int minor)
   snprintf(devname, sizeof(devname), DEV_FORMAT, minor);
   iinfo("Registering %s\n", devname);
 
-  ret = register_driver(devname, &g_tsdops, 0666, priv);
+  ret = register_driver(devname, &g_tsdops, 0600, priv);
   if (ret < 0)
     {
       ierr("ERROR: register_driver() failed: %d\n", ret);

arch/arm/src/stm32f7/stm32_bbsram.c

@@ -757,7 +757,7 @@ int stm32_bbsraminitialize(char *devpath, int *sizes)
     {
       snprintf(devname, sizeof(devname), "%s%d", devpath, i);
       ret = register_driver(devname, &g_stm32_bbsram_fops,
-                            0666, &g_bbsram[i]);
+                            0600, &g_bbsram[i]);
     }
 
   /* Disallow Access */

arch/arm/src/stm32h7/stm32_bbsram.c

@@ -828,7 +828,7 @@ int stm32_bbsraminitialize(char *devpath, int *sizes)
     {
       snprintf(devname, sizeof(devname), "%s%d", devpath, i);
       ret = register_driver(devname, &g_stm32_bbsram_fops,
-                            0666, &g_bbsram[i]);
+                            0600, &g_bbsram[i]);
     }
 
   /* Disallow Access */

arch/arm/src/tlsr82/tlsr82_spi_console.c

@@ -306,7 +306,7 @@ void spi_console_init(void)
 
   /* Driver register */
 
-  ret = register_driver("/dev/console", &g_consoleops, 0666, NULL);
+  ret = register_driver("/dev/console", &g_consoleops, 0600, NULL);
   if (ret < 0)
     {
       syslog(LOG_DEBUG, "register spi_console failed\n");

arch/renesas/src/rx65n/rx65n_sbram.c

@@ -662,7 +662,7 @@ int rx65n_sbraminitialize(char *devpath, int *sizes)
   for (i = 0; i < fcnt && ret >= OK; i++)
     {
       snprintf(devname, sizeof(devname), "%s%d", devpath, i);
-      ret = register_driver(devname, &g_rx65n_sbram_fops, 0666, &g_sbram[i]);
+      ret = register_driver(devname, &g_rx65n_sbram_fops, 0660, &g_sbram[i]);
     }
 
   /* Disallow Access */

arch/risc-v/src/common/espressif/esp_dedic_gpio.c

@@ -403,7 +403,7 @@ struct file *esp_dedic_gpio_new_bundle(
   dedic_gpio_common.refs++;
 
   register_driver(config->path, &dedic_gpio_ops,
-                  0666, priv);
+                  0600, priv);
   return (struct file *)priv;
 }
 

arch/risc-v/src/common/espressif/esp_lp_mailbox.c

@@ -322,6 +322,6 @@ int esp_lp_mailbox_init(void)
     }
 
   register_driver("/dev/lp_mailbox", esp_lp_mailbox_priv.ops,
-                  0666, (void *)&esp_lp_mailbox_priv);
+                  0600, (void *)&esp_lp_mailbox_priv);
   return OK;
 }

arch/risc-v/src/common/espressif/esp_nxdiag.c

@@ -306,7 +306,7 @@ static ssize_t esp_nxdiag_read(struct file *filep,
 
 static int esp_nxdiag_register(void)
 {
-  return register_driver("/dev/nxdiag", &g_nxdiagops, 0444, NULL);
+  return register_driver("/dev/nxdiag", &g_nxdiagops, 0440, NULL);
 }
 
 /****************************************************************************

arch/risc-v/src/common/espressif/esp_temperature_sensor.c

@@ -417,7 +417,7 @@ static void esp_temp_sensor_register(struct esp_temp_priv_s *priv)
 {
 #ifndef CONFIG_ESPRESSIF_TEMP_UORB
   register_driver(CONFIG_ESPRESSIF_TEMP_PATH, &g_esp_temp_sensor_fops,
-                  0666, priv);
+                  0660, priv);
 #else
   priv->lower.type = SENSOR_TYPE_TEMPERATURE;
   sensor_register(&priv->lower, CONFIG_ESPRESSIF_TEMP_PATH_DEVNO);

arch/risc-v/src/common/espressif/esp_ulp.c

@@ -423,7 +423,7 @@ static int esp_ulp_write(struct file *filep,
 
 static void esp_ulp_register(void)
 {
-  register_driver("/dev/ulp", &g_esp_ulp_fops, 0666, NULL);
+  register_driver("/dev/ulp", &g_esp_ulp_fops, 0660, NULL);
 }
 
 /****************************************************************************

arch/x86/src/qemu/qemu_keypad.c

@@ -303,5 +303,5 @@ static ssize_t keypad_read(struct file *filep, char *buf, size_t buflen)
 
 void qemu_keypad(void)
 {
-  register_driver("/dev/keypad", &g_keypadops, 0444, NULL);
+  register_driver("/dev/keypad", &g_keypadops, 0440, NULL);
 }

arch/x86/src/qemu/qemu_vga.c

@@ -588,7 +588,7 @@ int qemu_vga(void)
       return ret;
     }
 
-  register_driver("/dev/lcd", &g_vgaops, 0666, NULL);
+  register_driver("/dev/lcd", &g_vgaops, 0660, NULL);
 
   return OK;
 }

arch/xtensa/src/common/espressif/esp_nxdiag.c

@@ -313,7 +313,7 @@ static ssize_t esp_nxdiag_read(struct file *filep,
 
 static int esp_nxdiag_register(void)
 {
-  return register_driver("/dev/nxdiag", &g_nxdiagops, 0444, NULL);
+  return register_driver("/dev/nxdiag", &g_nxdiagops, 0440, NULL);
 }
 
 /****************************************************************************

arch/xtensa/src/common/espressif/esp_ulp.c

@@ -173,7 +173,7 @@ static int esp_ulp_write(struct file *filep,
 
 static void esp_ulp_register(void)
 {
-  register_driver("/dev/ulp", &g_esp_ulp_fops, 0666, NULL);
+  register_driver("/dev/ulp", &g_esp_ulp_fops, 0600, NULL);
 }
 
 /****************************************************************************

arch/xtensa/src/esp32/esp32_himem.c

@@ -249,7 +249,7 @@ int esp_himem_init(void)
 
   /* Register the character driver */
 
-  ret = register_driver("/dev/himem", &g_himemfops, 0666, NULL);
+  ret = register_driver("/dev/himem", &g_himemfops, 0600, NULL);
   if (ret < 0)
     {
       merr("ERROR: Failed to register driver: %d\n", ret);

arch/xtensa/src/esp32/esp32_himem_chardev.c

@@ -324,7 +324,7 @@ int himem_chardev_register(char *name, size_t size)
       return ret;
     }
 
-  ret = register_driver(dev->name, &g_fops, 0666, dev);
+  ret = register_driver(dev->name, &g_fops, 0600, dev);
   if (ret != 0)
     {
       merr("Failed to register driver. dev=%s\n", dev->name);

arch/xtensa/src/esp32s3/esp32s3_himem.c

@@ -1070,7 +1070,7 @@ int esp_himem_init(void)
 
   /* Register the character driver */
 
-  ret = register_driver("/dev/himem", &g_himemfops, 0666, NULL);
+  ret = register_driver("/dev/himem", &g_himemfops, 0600, NULL);
   if (ret < 0)
     {
       merr("ERROR: Failed to register driver: %d\n", ret);

audio/audio.c

@@ -1863,7 +1863,7 @@ int audio_register(FAR const char *name, FAR struct audio_lowerhalf_s *dev)
   dev->priv = upper;
 
   audinfo("Registering %s\n", path);
-  return register_driver(path, &g_audioops, 0666, upper);
+  return register_driver(path, &g_audioops, 0660, upper);
 }
 
 #endif /* CONFIG_AUDIO */

boards/arm/cxd56xx/drivers/sensors/ak09912_scu.c

@@ -556,7 +556,7 @@ int ak09912_scu_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_ak09912fops, 0666, priv);
+  ret = register_driver(path, &g_ak09912fops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);

boards/arm/cxd56xx/drivers/sensors/apds9930_scu.c

@@ -974,7 +974,7 @@ int apds9930als_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_apds9930alsfops, 0666, priv);
+  ret = register_driver(path, &g_apds9930alsfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);
@@ -1028,7 +1028,7 @@ int apds9930ps_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_apds9930psfops, 0666, priv);
+  ret = register_driver(path, &g_apds9930psfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);

boards/arm/cxd56xx/drivers/sensors/bh1721fvc_scu.c

@@ -376,7 +376,7 @@ int bh1721fvc_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_bh1721fvcfops, 0666, priv);
+  ret = register_driver(path, &g_bh1721fvcfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);

boards/arm/cxd56xx/drivers/sensors/bh1745nuc_scu.c

@@ -510,7 +510,7 @@ int bh1745nuc_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_bh1745nucfops, 0666, priv);
+  ret = register_driver(path, &g_bh1745nucfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);

boards/arm/cxd56xx/drivers/sensors/bm1383glv_scu.c

@@ -541,7 +541,7 @@ int bm1383glv_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_bm1383glvfops, 0666, priv);
+  ret = register_driver(path, &g_bm1383glvfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);

boards/arm/cxd56xx/drivers/sensors/bm1422gmv_scu.c

@@ -525,7 +525,7 @@ int bm1422gmv_register(const char *devpath, int minor,
   /* Register the character driver */
 
   snprintf(path, sizeof(path), "%s%d", devpath, minor);
-  ret = register_driver(path, &g_bm1422gmvfops, 0666, priv);
+  ret = register_driver(path, &g_bm1422gmvfops, 0660, priv);
   if (ret < 0)
     {
       snerr("Failed to register driver: %d\n", ret);