Merge pull request #9727 from jepler/fix-assign-wifi-radio-dns-raspi

dhalbert · web-flow · commit aa385486b43f · 2024-10-18T10:17:11.000-04:00
Fix crash on assignment of wifi.radio.dns on raspberrypi boards
diff --git a/ports/raspberrypi/common-hal/wifi/Radio.c b/ports/raspberrypi/common-hal/wifi/Radio.c
@@ -574,7 +574,7 @@ void common_hal_wifi_radio_set_dns(wifi_radio_obj_t *self, mp_obj_t dns_addrs_ob
         addr.addr = IPADDR_NONE;
     } else {
         mp_obj_t dns_addr_obj = mp_obj_subscr(dns_addrs_obj, MP_OBJ_NEW_SMALL_INT(0), MP_OBJ_SENTINEL);
-        socketpool_resolve_host_raise(dns_addr_obj, &addr);
+        socketpool_resolve_host_raise(mp_obj_str_get_str(dns_addr_obj), &addr);
     }
     dns_setserver(0, &addr);
 }

Original file line number	Diff line number	Diff line change
`@@ -574,7 +574,7 @@ void common_hal_wifi_radio_set_dns(wifi_radio_obj_t *self, mp_obj_t dns_addrs_ob`
`574`	`574`	`addr.addr = IPADDR_NONE;`
`575`	`575`	`} else {`
`576`	`576`	`mp_obj_t dns_addr_obj = mp_obj_subscr(dns_addrs_obj, MP_OBJ_NEW_SMALL_INT(0), MP_OBJ_SENTINEL);`
`577`		`- socketpool_resolve_host_raise(dns_addr_obj, &addr);`
	`577`	`+ socketpool_resolve_host_raise(mp_obj_str_get_str(dns_addr_obj), &addr);`
`578`	`578`	`}`
`579`	`579`	`dns_setserver(0, &addr);`
`580`	`580`	`}`