diff options
| author | David Lin <[email protected]> | 2020-03-08 00:48:32 +0800 |
|---|---|---|
| committer | GitHub <[email protected]> | 2020-03-08 00:48:32 +0800 |
| commit | c689c2c67fa5519b510fe22b7f8fe61b4aa08b40 (patch) | |
| tree | dad1da47726d561834fe9f714555993a686149b0 | |
| parent | 9d9432eb93460284c0b518900bce227f3310138f (diff) | |
Fixed a hidden stack overflow bug
'memset' should been next step after 'NULL' judge
| -rw-r--r-- | bsp/fh8620/drivers/pwm.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/bsp/fh8620/drivers/pwm.c b/bsp/fh8620/drivers/pwm.c index a9d2645631..ff02a5a6db 100644 --- a/bsp/fh8620/drivers/pwm.c +++ b/bsp/fh8620/drivers/pwm.c @@ -186,12 +186,13 @@ int fh_pwm_probe(void *priv_data) PWM_Enable(pwm_obj, RT_FALSE); pwm_dev = rt_malloc(sizeof(struct rt_device)); - rt_memset(pwm_dev, 0, sizeof(struct rt_device)); if (pwm_dev == RT_NULL) { rt_kprintf("ERROR: %s rt_device malloc failed\n", __func__); } + + rt_memset(pwm_dev, 0, sizeof(struct rt_device)); pwm_dev->user_data = &pwm_drv; pwm_dev->open =fh_pwm_open; |
