extmod/modwebrepl: Get rid of using strncpy().

pfalcon · pfalcon · commit b8468d12a1a3 · 2016-05-02T20:52:34.000+03:00
diff --git a/extmod/modwebrepl.c b/extmod/modwebrepl.c
@@ -285,8 +285,11 @@ STATIC mp_uint_t webrepl_write(mp_obj_t self_in, const void *buf, mp_uint_t size
 }
 
 STATIC mp_obj_t webrepl_set_password(mp_obj_t passwd_in) {
-    const char *passwd = mp_obj_str_get_str(passwd_in);
-    strncpy(webrepl_passwd, passwd, sizeof(webrepl_passwd));
+    mp_uint_t len;
+    const char *passwd = mp_obj_str_get_data(passwd_in, &len);
+    len = MIN(len, sizeof(webrepl_passwd) - 1);
+    memcpy(webrepl_passwd, passwd, len);
+    webrepl_passwd[len] = 0;
     return mp_const_none;
 }
 STATIC MP_DEFINE_CONST_FUN_OBJ_1(webrepl_set_password_obj, webrepl_set_password);

Original file line number	Diff line number	Diff line change
`@@ -285,8 +285,11 @@ STATIC mp_uint_t webrepl_write(mp_obj_t self_in, const void *buf, mp_uint_t size`
`285`	`285`	`}`
`286`	`286`
`287`	`287`	`STATIC mp_obj_t webrepl_set_password(mp_obj_t passwd_in) {`
`288`		`- const char *passwd = mp_obj_str_get_str(passwd_in);`
`289`		`- strncpy(webrepl_passwd, passwd, sizeof(webrepl_passwd));`
	`288`	`+ mp_uint_t len;`
	`289`	`+ const char *passwd = mp_obj_str_get_data(passwd_in, &len);`
	`290`	`+ len = MIN(len, sizeof(webrepl_passwd) - 1);`
	`291`	`+ memcpy(webrepl_passwd, passwd, len);`
	`292`	`+ webrepl_passwd[len] = 0;`
`290`	`293`	`return mp_const_none;`
`291`	`294`	`}`
`292`	`295`	`STATIC MP_DEFINE_CONST_FUN_OBJ_1(webrepl_set_password_obj, webrepl_set_password);`