Fixed incorrect usage of memset() in PWM code which leads to incomplete initialization of struct. Thanks to GCC warning -Wsizeof-pointer-memaccess.

Родион Квашнин · igrr · commit 9791a48d4ae6 · 2017-01-19T20:18:31.000+08:00
diff --git a/cores/esp8266/core_esp8266_wiring_pwm.c b/cores/esp8266/core_esp8266_wiring_pwm.c
@@ -187,7 +187,7 @@ extern void __analogWrite(uint8_t pin, int value)
     }
     if((pwm_mask & (1 << pin)) == 0) {
         if(pwm_mask == 0) {
-            memset(&_pwm_isr_data, 0, sizeof(struct pwm_isr_data*));
+            memset(&_pwm_isr_data, 0, sizeof(_pwm_isr_data));
             start_timer = true;
         }
         pinMode(pin, OUTPUT);

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ extern void __analogWrite(uint8_t pin, int value)`
`187`	`187`	`}`
`188`	`188`	`if((pwm_mask & (1 << pin)) == 0) {`
`189`	`189`	`if(pwm_mask == 0) {`
`190`		`- memset(&_pwm_isr_data, 0, sizeof(struct pwm_isr_data*));`
	`190`	`+ memset(&_pwm_isr_data, 0, sizeof(_pwm_isr_data));`
`191`	`191`	`start_timer = true;`
`192`	`192`	`}`
`193`	`193`	`pinMode(pin, OUTPUT);`