テストパターンを作成したいです

Question

テストパターンを作成したいです。
作成したのは、inet_ntopなのですが、様々なネットワークアドレス構造体についてそれが変換可能なものかをテストしたいです。
下記は、inet_ntopのmanページです。
今回はAF_INETの場合を考えます。
https://linuxjm.osdn.jp/html/LDP_man-pages/man3/inet_ntop.3.html

具体的には、
"130.0.7.23"はOKですが、
"130.00.7.23"や"130.01.7.23"はNGというふうにテストしたいです。
mainでテストパターンを作成しましたが、一つしか作成できません。
原因として、もし2つ以上つくってしまうと変数の中身が上書きされてしまうというのは分かっています。
ですが、どうすればテストパターンをいくつも作成することができるのか分かりません。

あと、本題とはそれる事なのですが、a.s_addr = inet_addr("127.256.0.");というふうに、エラーとなる場合でコンパイルして実行すると、
255.255.255.255と表示されます。
私として、エラー表示がされるよう作成したつもりなので、なぜこのように表示されるのか分かりません。

大変恐縮ではございますがお力添えいただければ幸いです。
宜しくお願い致します。

#include <stdio.h>
#include <string.h>
#include <errno.h>
#include <arpa/inet.h>
#define p ( ( const unsigned char* )src )

const char* inet_ntop( const void* src, char* dst, socklen_t cnt ){

    // 変数の宣言
    char buf[sizeof "255.255.255.255"];

    // srcの数値を文字列に変換
    const unsigned n = 1 + sprintf( buf, "%u.%u.%u.%u", p[0], p[1], p[2], p[3] );

    // 文字列がdstのサイズを超えているか判定
    if ( cnt < n ) {
        errno = ENOSPC;
        return( NULL );

    // 変換した文字列をdstにコピー
    }else{

    return( ( const char* )memcpy( dst, buf, n ) );
    }
}

int main(){
    struct in_addr a;
    char dst[256];

    a.s_addr = inet_addr( "130.0.7.23" );

    if ( inet_ntop( &a, dst, sizeof dst ) ) {
    printf( "%s\n", dst );
    return( 0 );
    }else{
        return( 1 );
    }
}

Comments

[y_waiwai]

あなたのいうテストパターンというのは何でしょうか。
意味不明です

[ht3433]

コメントしていただき、ありがとうございます。
ただいま修正致します。

[SHOMI]

>255.255.255.255と表示されます
理由はmanページに書いてありますよ
https://linuxjm.osdn.jp/html/LDP_man-pages/man3/inet_aton.3.html
inet_addr() 関数は、インターネットホストのアドレス cp を、 IPv4 の数値とドットによる表記からネットワークバイトオーダでの バイナリ値へ変換して返す。 入力が不正な場合、 INADDR_NONE (普通は -1) を返す。 -1 は有効なアドレス (255.255.255.255) なので、この関数を使うと 問題になるかもしれない。

[ht3433]

コメントしていただき、ありがとうございます。
なるほど、そういうことだったんですね。
勉強になります。

[ht3433]

y_waiwaiさん
先ほど、質問の修正をし、テストパターンについて記述いたしました。

[SHOMI]

>もし2つ以上つくってしまうと変数の中身が上書きされてしまう
テスト通過後次の結果で上書きされても問題ないと思いますが

[ht3433]

コメントしていただき、ありがとうございます。
このソースコードだと、テスト通過する前に上書きされると思うのですが、
違うのでしょうか？

[SHOMI]

>テスト通過する前に上書きされると思うのですが
テスト一つしかありませんが、どこのことを言われていますか？

[ht3433]

ソースコードには１つしかありませんが、例えば、
a.s_addr = inet_addr( "130.0.7.23" );
a.s_addr = inet_addr( "130.1.7.23" );
とした場合、a.s_addrの中身は "130.1.7.23"に上書きされるので、"130.0.7.23"のテストができないということです。
説明不足ですみません。

[SHOMI]

"130.0.7.23"についてのテストが完了してから
a.s_addr = inet_addr( "130.1.7.23" );
を実行すればよいだけでは？

[ht3433]

つまり、下記のようにすればよいということでしょうか？
a.s_addr = inet_addr( "130.0.7.23" );

if ( inet_ntop( &a, dst, sizeof dst ) ) {
printf( "%s\n", dst );

}else{
return( 1 );
}
a.s_addr = inet_addr( "130.1.7.23" );

if ( inet_ntop( &a, dst, sizeof dst ) ) {
printf( "%s\n", dst );
return( 0 );
}else{
return( 1 );
}

[SHOMI]

そうです。
今は数が少ないので問題ないでしょうが、dstの中に意図したとおりの結果が格納されているかのチェックもprintfによる目視ではなく、プログラム側でチェックした方がよいと思います。
あと、main()でreturnすると終わってしまうので、別関数にするなり不合格を示すprintf等に置き換えるなりして下さい。

[ht3433]

コメントしていただき、ありがとうございます。
確かに、目視よりプログラム側でチェックした方がいいですよね。
＞あと、main()でreturnすると終わってしまう
これはなにがいけないのでしょうか？下記のコードのようにしても問題ないと思うですがどうでしょうか？あと、テストパターンの作成を2つ以上作ることに関しては、一応できました。
何かアドバイスをいただけるとありがたいです。
ソースコード
int main(){
struct in_addr a;
char dst[256];

a.s_addr = inet_addr( "130.0.7.23" );
assert( a.s_addr != INADDR_NONE );

a.s_addr = inet_addr( "256.0.7.23" );
assert( a.s_addr != INADDR_NONE );

inet_ntop( &a, dst, sizeof dst );
return( 0 );

[SHOMI]

>これはなにがいけないのでしょうか
一件でもNGとなればその時点で終了するという方針なのであれば問題ありません。

[ht3433]

コメントしていただき、ありがとうございます。
一件でもNGとなればその時点で終了するという方針でやっていたので、よかったです。
ご指摘ありがとうございました。

[SHOMI]

新しいコードはinet_addrの結果のチェックで、inet_ntopのテストをしていませんよ…

Answer 1

#include <stdio.h>
#include <string.h>
#include <errno.h>
#include <arpa/inet.h>
#define p ( ( const unsigned char* )src )

const char* inet_ntop( const void* src, char* dst, socklen_t cnt ){

    // 変数の宣言
    char buf[sizeof "255.255.255.255"];

    // srcの数値を文字列に変換
    const unsigned n = 1 + sprintf( buf, "%u.%u.%u.%u", p[0], p[1], p[2], p[3] );

    // 文字列がdstのサイズを超えているか判定
    if ( cnt < n ) {
        errno = ENOSPC;
        return( NULL );

    // 変換した文字列をdstにコピー
    }else{

    return( ( const char* )memcpy( dst, buf, n ) );
    }
}

int main(){
struct in_addr a;
char dst[256];

a.s_addr = inet_addr( "130.0.7.23" );
assert( a.s_addr != INADDR_NONE );

a.s_addr = inet_addr( "256.0.7.23" );
assert( a.s_addr != INADDR_NONE );

inet_ntop( &a, dst, sizeof dst );
return( 0 ); 
}

Answer 2

様々なネットワークアドレス構造体

と，それらがOKかNGかを表すデータを（配列なり何なりで）用意して，inet_ntop()の結果と照らし合わせるループでも書けばよいのではないでしょうか．

Answer 3

以下のようなテキストファイルを作成し、それを1行ごとに処理されてはいかがでしょうか。

130.0.7.23
130.00.7.23
888.999.333.000

int main()
{
    struct in_addr a;
    char dst[256];
    FILE *fp;
    char buff[256];
    int  len;
    in_addr_t ret;
    fp = fopen("test.txt","r");
    if (fp == NULL){
        printf("file open error\n");
        return(10);
    }
    while(1){
        memset(buff,0x00,sizeof(buff));
        if (fgets(buff,sizeof(buff)-1,fp) == NULL) break;
        len = strlen(buff);
        buff[len-1] = '\0';   //改行の削除
        printf("data=<%s>\n",buff);
        a.s_addr = inet_addr(buff);

        if (inet_ntop(&a, dst, sizeof dst)) {
            printf("%s\n", dst);
        } else {
            printf("inet_ntop error\n");
        }
    }
    fclose(fp);
    return(0);
}

Comments

[ht3433]

回答していただき、ありがとうございます。
tatau99さんがおっしゃっているのは下記のように書けばよいということでしょうか？
a.s_addr = inet_addr( "130.0.7.23" );

if ( inet_ntop( &a, dst, sizeof dst ) ) {
printf( "%s\n", dst );

}else{
return( 1 );
}
a.s_addr = inet_addr( "130.00.7.23" );

if ( inet_ntop( &a, dst, sizeof dst ) ) {
printf( "%s\n", dst );
return( 0 );
}else{
return( 1 );
}

[tatsu99]

mainのみかいてみました。
ファイル名は"test.txt"であることが前提です。