mirror of https://github.com/ARMmbed/mbed-os.git
				
				
				
			connectivity tests: avoid crash with null pointer
							parent
							
								
									756830e776
								
							
						
					
					
						commit
						79ae89f740
					
				| 
						 | 
				
			
			@ -168,6 +168,7 @@ static void net_bringup()
 | 
			
		|||
    MBED_ASSERT(MBED_CONF_APP_DNS_TEST_HOSTS_NUM >= MBED_CONF_NSAPI_DNS_CACHE_SIZE && MBED_CONF_APP_DNS_TEST_HOSTS_NUM >= MBED_CONF_APP_DNS_SIMULT_QUERIES + 1);
 | 
			
		||||
 | 
			
		||||
    net = NetworkInterface::get_default_instance();
 | 
			
		||||
    TEST_ASSERT_NOT_NULL_MESSAGE(net, "No NetworkInterface configured");
 | 
			
		||||
    nsapi_error_t err = net->connect();
 | 
			
		||||
    TEST_ASSERT_EQUAL(NSAPI_ERROR_OK, err);
 | 
			
		||||
    SocketAddress address;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -75,6 +75,7 @@ nsapi_version_t get_ip_version()
 | 
			
		|||
static void _ifup()
 | 
			
		||||
{
 | 
			
		||||
    NetworkInterface *net = NetworkInterface::get_default_instance();
 | 
			
		||||
    TEST_ASSERT_NOT_NULL_MESSAGE(net, "No NetworkInterface configured");
 | 
			
		||||
    nsapi_error_t err = net->connect();
 | 
			
		||||
    TEST_ASSERT_EQUAL(NSAPI_ERROR_OK, err);
 | 
			
		||||
    SocketAddress address;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -71,6 +71,7 @@ void drop_bad_packets(TLSSocket &sock, int orig_timeout)
 | 
			
		|||
static void _ifup()
 | 
			
		||||
{
 | 
			
		||||
    NetworkInterface *net = NetworkInterface::get_default_instance();
 | 
			
		||||
    TEST_ASSERT_NOT_NULL_MESSAGE(net, "No NetworkInterface configured");
 | 
			
		||||
    nsapi_error_t err = net->connect();
 | 
			
		||||
    TEST_ASSERT_EQUAL(NSAPI_ERROR_OK, err);
 | 
			
		||||
    SocketAddress address;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -62,6 +62,7 @@ void drop_bad_packets(UDPSocket &sock, int orig_timeout)
 | 
			
		|||
static void _ifup()
 | 
			
		||||
{
 | 
			
		||||
    NetworkInterface *net = NetworkInterface::get_default_instance();
 | 
			
		||||
    TEST_ASSERT_NOT_NULL_MESSAGE(net, "No NetworkInterface configured");
 | 
			
		||||
    nsapi_error_t err = net->connect();
 | 
			
		||||
    TEST_ASSERT_EQUAL(NSAPI_ERROR_OK, err);
 | 
			
		||||
    SocketAddress address;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -75,6 +75,7 @@ void NETWORKINTERFACE_STATUS()
 | 
			
		|||
    current_status = NSAPI_STATUS_ERROR_UNSUPPORTED;
 | 
			
		||||
 | 
			
		||||
    net = NetworkInterface::get_default_instance();
 | 
			
		||||
    TEST_ASSERT_NOT_NULL_MESSAGE(net, "No NetworkInterface configured");
 | 
			
		||||
    net->attach(status_cb);
 | 
			
		||||
    net->set_blocking(true);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -30,6 +30,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    char ssid[SSID_MAX_LEN + 1] = MBED_CONF_APP_WIFI_UNSECURE_SSID;
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_disconnect_repeat(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
    nsapi_error_t error;
 | 
			
		||||
 | 
			
		||||
    error = wifi->set_credentials(MBED_CONF_APP_WIFI_SECURE_SSID, MBED_CONF_APP_WIFI_PASSWORD, get_security());
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -26,6 +26,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_nocredentials(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
    nsapi_error_t error_connect, error_disconnect;
 | 
			
		||||
    error_connect = wifi->connect();
 | 
			
		||||
    error_disconnect = wifi->disconnect();
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_params_channel(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (wifi->set_channel(1) == NSAPI_ERROR_UNSUPPORTED && wifi->set_channel(36) == NSAPI_ERROR_UNSUPPORTED) {
 | 
			
		||||
        TEST_IGNORE_MESSAGE("set_channel() not supported");
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_params_channel_fail(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (wifi->set_channel(1) == NSAPI_ERROR_UNSUPPORTED && wifi->set_channel(36) == NSAPI_ERROR_UNSUPPORTED) {
 | 
			
		||||
        TEST_IGNORE_MESSAGE("set_channel() not supported");
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -27,6 +27,10 @@ void wifi_connect_params_null(void)
 | 
			
		|||
{
 | 
			
		||||
    nsapi_error_t error;
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
    error = wifi->connect(NULL, NULL);
 | 
			
		||||
    wifi->disconnect();
 | 
			
		||||
    TEST_ASSERT_EQUAL(NSAPI_ERROR_PARAMETER, error);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_params_valid_secure(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (wifi->connect(MBED_CONF_APP_WIFI_SECURE_SSID, MBED_CONF_APP_WIFI_PASSWORD, get_security()) == NSAPI_ERROR_OK) {
 | 
			
		||||
        if (wifi->disconnect() == NSAPI_ERROR_OK) {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_secure(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    // Driver shall cache the credentials
 | 
			
		||||
    char ssid[] = MBED_CONF_APP_WIFI_SECURE_SSID;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_connect_secure_fail(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->set_credentials(MBED_CONF_APP_WIFI_SECURE_SSID, "aaaaaaaa", get_security()));
 | 
			
		||||
    nsapi_error_t error;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -28,6 +28,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_get_rssi(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->set_credentials(MBED_CONF_APP_WIFI_SECURE_SSID, MBED_CONF_APP_WIFI_PASSWORD, get_security()));
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -27,6 +27,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_scan(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    WiFiAccessPoint ap[MBED_CONF_APP_MAX_SCAN_SIZE];
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -26,5 +26,9 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_scan_null(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
    TEST_ASSERT(wifi->scan(NULL, 0) >= 1);
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -30,6 +30,10 @@ void wifi_set_channel(void)
 | 
			
		|||
    bool is_5Ghz = false;
 | 
			
		||||
 | 
			
		||||
    WiFiInterface *wifi = get_interface();
 | 
			
		||||
    TEST_ASSERT(wifi);
 | 
			
		||||
    if (wifi == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (wifi->set_channel(1) == NSAPI_ERROR_UNSUPPORTED && wifi->set_channel(36) == NSAPI_ERROR_UNSUPPORTED) {
 | 
			
		||||
        TEST_IGNORE_MESSAGE("set_channel() not supported");
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -26,6 +26,10 @@ using namespace utest::v1;
 | 
			
		|||
void wifi_set_credential(void)
 | 
			
		||||
{
 | 
			
		||||
    WiFiInterface *iface = get_interface();
 | 
			
		||||
    TEST_ASSERT(iface);
 | 
			
		||||
    if (iface == NULL) {
 | 
			
		||||
        return;
 | 
			
		||||
    }
 | 
			
		||||
    nsapi_error_t error;
 | 
			
		||||
 | 
			
		||||
    error = iface->set_credentials(NULL, NULL, NSAPI_SECURITY_NONE);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue