void型をuint32_t型にキャストしたい

Question

前提・実現したいこと

ネットで調べながらコーディングしてみたのですが、void型をuint32_t型にキャストしたいのですがエラーになってしまい、どう対処すればよいか困っています。
最終的にはLinuxコマンドのinet_ptonを作るつもりです。
お力添えいただければ幸いです。
宜しくお願い致します。

発生している問題・エラーメッセージ

エラー: ‘uint32_t {aka unsigned int}’ から ‘const void*’ へ の無効な変換です [-fpermissive]
     memcpy( dst, No.number1, 4);

エラー:   initializing argument 2 of ‘void* memcpy(void*, const void*, size_t)’ [-fpermissive]
 extern void *memcpy (void *__restrict __dest, const void *__restrict __src,

該当のソースコード

#include <assert.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>


int inet_pton(const char *src, void *dst ){

    // 変数の宣言
    int suuti1,suuti2,suuti3,suuti4;

    // 共用体の型の定義
    union number{
        uint32_t number1;
        uint8_t number2[4];
    };

    // 共用体の変数名の宣言
    union number No;

    // srcの文字列を数値に変換する
    sscanf( src, "%d.%d.%d.%d", &suuti1, &suuti2, &suuti3, &suuti4 );

    // 数値に変換したものを変数に格納する
    No.number2[0] = suuti1;
    No.number2[1] = suuti2;
    No.number2[2] = suuti3;
    No.number2[3] = suuti4;

    // void型のdstをuint32_t型にキャストする
    No.number1 = ( uintptr_t )&dst;

    // number1の4バイト分のメモリをdstにコピーする
    memcpy( dst, No.number1, 4);


    return( 1 );
}

int main(){
    int result;
    struct in_addr in_addr;

    result = inet_pton("130.0.7.23", &in_addr);
    assert(result == 1 && in_addr.s_addr == ((23<<24)|(7<<16)|(0<<8)|130));

    result = inet_pton("130.00.7.23", &in_addr);
    assert(result == 0);

    result = inet_pton("130.01.7.23", &in_addr);
    assert(result == 0);

    return(0);
}

Comments

[tiitoi]

glibc はオープンソースなので、ソースを見てみればいいのではないでしょうか。
https://github.com/kraj/glibc/blob/master/resolv/inet_pton.c

[ht3433]

回答していただき、ありがとうございます。
早速ソースを拝見致しましたが、内容が難しく、あまり参考にならなかったです。
理解力に乏しく、大変申し訳ございません。

Answer 1

void型は uint32_t型に変換できません。
inet_pton はコマンドではなく、関数です。
inet_pton は <arpa/inet.h> で宣言されているので
C では同じ関数は定義できません。
my_inet_pton とでもしましょう。

次のように書けばよいでしょう。

#include <assert.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>


int my_inet_pton(const char *src, void *dst)
{
    int suuti1, suuti2, suuti3, suuti4; // 変数の宣言
    union number{ // 共用体の型の定義
        uint32_t number1;
        uint8_t number2[4];
    };
    union number No; // 共用体の変数名の宣言

    // srcの文字列を数値に変換する
    if (sscanf(src, "%d.%d.%d.%d", &suuti1, &suuti2, &suuti3, &suuti4) != 4)
		return 0;

    // 数値に変換したものを変数に格納する
    No.number2[0] = suuti1;
    No.number2[1] = suuti2;
    No.number2[2] = suuti3;
    No.number2[3] = suuti4;

    memcpy(dst, &No.number1, 4);
    return 1;
}

int main(void)
{
	unsigned char buf[4];
	int n = my_inet_pton("216.58.197.174", buf);
	printf("%d.%d.%d.%d\n", buf[0], buf[1], buf[2], buf[3]);
}

同じことは次のように書けます。

#include <assert.h>
#include <arpa/inet.h>
#include <stdio.h>

int my_inet_pton(const char *src, void *dst)
{
	char *p = (char *)dst;
    if (sscanf(src, "%hhd.%hhd.%hhd.%hhd", p, p+1, p+2, p+3) == 4)
		return 1;
	return 0;
}

int main(void)
{
	unsigned char buf[4];
	int n = my_inet_pton("216.58.197.174", buf);
	printf("%d.%d.%d.%d\n", buf[0], buf[1], buf[2], buf[3]);
}

---

追記
質問のコード

	// void型のdstをuint32_t型にキャストする
    No.number1 = ( uintptr_t )&dst;

コメントが間違っています。
dst の型は、void型ではなく、void *、すなわち「voidへのポインタ」型です。
&dst は引数 dst のアドレスです。
&dst の型は void * *、すなわち「voidへのポインタへのポインタ」型です。
(uintptr_t) により void * * を unsigned int * にキャストしています。
No.number1 の型は uint32_t、すなわち unsigned int です。
ポインタが 4バイトのコンパイラでは、No.number1 にはそのポインタの値が入るでしょう。
ポインタが 8バイトのコンパイラでは、No.number1 にはそのポインタの値の下位32ビットが入るでしょう。
union の意味をご存知でしょうか？
No は 4バイトしかありません。
そこには、No.number2[0]～No.number2[3] に代入した値が入っています。
No.number1 への代入は、.number2 に代入した値を破壊します。
引数 dst のアドレスが No.number1 には代入されたからです。

質問のコード

// number1の4バイト分のメモリをdstにコピーする
    memcpy( dst, No.number1, 4);

memcpy の第2引数には、コピー元のアドレスを渡さなければなりません。
したがって、memcpy( dst, &No.number1, 4); としないといけません。
No.number1 = ( uintptr_t )&dst; を削除すれば、
No.number1 には .number2 に代入された値が既に入っているので、
memcpy によって目的が達成できます。

Comments

[ht3433]

回答していただき、ありがとうございます。
返信が遅れてしまい、申し訳ございません。
とても参考になり、おかげさまでなんとかうまくいきました。
本当にありがとうございました！

Answer 2

以下のようにドットで区切られた4つの数値を読んでいって、数値に変換すればいいのではないでしょうか。

#include <stdio.h>
#include <ctype.h>

/**
 * @brief IPv4 アドレスを表す文字列をバイナリに変換する。
 * 
 * @param src IPv4 アドレスを表す文字列 (例 "192.168.1.11")
 * @param dst 変換結果
 */
void inet_pton(const char *src, void *dst)
{
    int i, j, v;
	unsigned char *d = (unsigned char *)dst;

    for (i = 0; i < 4; ++i) {
        v = 0;  // 文字列を変換した数値

        // . で区切られた数字1つ分を読み込み、数値に変換する。
        for (j = 0; j < 3 && isdigit(src[j]); ++j)
            v = 10 * v + src[j] - '0';  // 整数での値

        d[i] = v;
        src += j + 1;
    }
}

int main()
{
    char *src = "192.168.1.11";
    unsigned char dst[4];

    inet_pton(src, (void*)dst);

    // 192 168 11 1
    printf("%hhu %hhu %hhu %hhu\n", dst[0], dst[1], dst[2], dst[3]);
}

Wandbox の C99環境で動作確認

Answer 3

型が違うものの代入ができないといってますね
そこはどうしたい（どうなってほしい）んでしょうか

Comments

[ht3433]

回答していただき、ありがとうございます。
文字列srcをIPｖ4のネットワークアドレス構造体に変換したいです。
また、変換後のsrcのアドレスをstruct in_addrに変換させ、dstにコピーしたいです。

[y_waiwai]

なら、文字列をピリオドで分割して整数変換しましょう

[ht3433]

回答していただき、ありがとうございます。
助言を受け、コーディングしてみたのですが、行き詰ってしまいました。
もう少し具体的なアドバイスをいただけるとありがたいです。

ソースコード
#include <assert.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int inet_pton(const char *src,void *dst )
{

union number{
uint32_t number1;
uint8_t number2[4];
};
union number No;
uint32_t net_No;
int i = 0;

// ドットを区切りに文字列を分割
// 1回目
src = strtok( number2, ".");
No.number2[0] = src;

// 2回目以降
while( src != NULL ){
// strtok関数により変更されたNULLのポインタが先頭
src = strtok( NULL, ".");

// srcがNULLの場合、エラーが発生するので対処
if( src != NULL ){
i++;
No.number2[i] = src;
}
}

net_No = htonl( No.number1 );

// srcをIPｖ4のネットワークアドレス構造体に変換
src = net_No;

in_addr = src;

// dstにコピーする
strcpy( dst, in_addr );

return( 0 );
}

int main(){
int result;
struct in_addr in_addr;
result = inet_pton("130.0.7.23", &in_addr);
assert(result == 1 && in_addr.s_addr == ((23<<24)|(7<<16)|(0<<8)|130));

result = inet_pton("130.00.7.23", &in_addr);
assert(result == 0);

result = inet_pton("130.01.7.23", &in_addr);
assert(result == 0);

return(0);
}

[y_waiwai]

なんかの課題かなんかだと思いますが。。
・strtokの戻り値は文字列（のアドレス）ですんで、これを整数に変換する必要があります(atoi関数？)
・IPの文字列がどこにはいってるのか、変換してどの変数に入れるのか見直しなさい。このコードではムチャムチャです
・んで、strtok関数というのは元の文字列を破壊します。入力文字列をいったん別の配列にコピーしてから変換しましょう

[ht3433]

アドバイスをしていただき、ありがとうございます。
確かにコードが無茶苦茶だったので、一度見直し、一つずつ処理を考えていこうと思います。
そこで、srcをネットワーク構造体に変換する処理がうまくいってないのですが、そのエラー内容が、
エラー: ‘uint32_t {aka unsigned int}’ から ‘const char*’ への無効な変換です [-fpermissive]
src = net_No;
となっており、変換ができないか調べてみたのですが、解決できませんでした。
どのようにすればよいのでしょうか？
宜しくお願い致します。

ソースコード

#include <assert.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int inet_pton( const char *src,void *dst ){

// 共用体の型の定義
union number{
uint32_t number1;
uint8_t number2[4];
};

// 共用体の変数名の宣言
union number No;
uint32_t net_No;

// アドレスを変数に代入
No.number2[0] = 130;
No.number2[1] = 0;
No.number2[2] = 7;
No.number2[3] = 23;

// ネットワークバイトオーダーをポストバイトオーダーに変換
net_No = htonl( No.number1 );

// srcをネットワーク構造体に変換する
src = net_No;

return( 0 );
}

int main(){
int result;
struct in_addr in_addr;

result = inet_pton("130.0.7.23", &in_addr);
assert(result == 1 && in_addr.s_addr == ((23<<24)|(7<<16)|(0<<8)|130));

return(0);
}

[y_waiwai]

printf("%d.%d.%d.%d",No.number2[0],No.number2[1],No.number2[2],No.number2[3]);

こういうことがしたいのかな？

[ht3433]

いえ、私がしたいのは、文字列srcをネットワークアドレス構造体に変換させたいのであって、取得した文字列を出力することではないです。

[y_waiwai]

srcを変換したいなら、srcに代入してる場合じゃないでしょうｗ

[ht3433]

すみません、よくわからないです。
srcを変換させたいから、net_Noの中身をsrcに代入するのではないでしょうか？

[y_waiwai]

関数のなかの各行の命令がどういう動作をしていき、各変数のナカミがどうなるかを考えてみましょう。

どうもC言語の基礎からやりなおしてもらったほうがよろしいかと。

[ht3433]

y_waiwaiさんのおっしゃるとおり、命令がどういう動きをして、各変数の中身も考えてみたのですが、どうもよく分かりません。
なにかヒントでもいただければ幸いです。

[y_waiwai]

src = net_No;
という命令文は、net_No のナカミを src にコピーする、という動作になります
しかし、双方の変数の型が違うので、コピーしても意味がないよ、とコンパイラさんが知らせてくれています
ムリヤリ実行したところで意味のあるもんにはならないです
あなたの言う通り、srcからなにかに変更したいと言うなら、
net_No = 変換関数(src);
という命令文となる、はず、です

result = inet_pton("130.0.7.23", &in_addr);

これはそうなってるでしょう

[ht3433]

ヒントをいただき、ありがとうございます。
atoi関数を使用して、文字列srcを数値に変換しましたが、y_waiwaiさんがおっしゃりたいことはこういうことでしょうか？
// srcをネットワーク構造体に変換する
net_No = atoi( src );

[y_waiwai]

まあしかし、atoi関数では、あなたの思うような変換はできないですよね。

[ht3433]

それはどういうことでしょうか？
詳しく教えていただきたいです。

[y_waiwai]

http://hitorilife.com/atoi.php

// srcをネットワーク構造体に変換する
というような関数ではない、ということです

[ht3433]

なるほど、確かにおっしゃるとおりですね。
では、sscanf関数はどうでしょうか？
http://www.hitorilife.com/sscanf.php

[y_waiwai]

sscanf関数を使えばそういう動作をする関数は作れるでしょう。

[ht3433]

回答していただき、ありがとうございます。
sscanfを使って、srcをネットワーク構造体に変換しようと思ったのですが、うまくいきません。
警告: 書式 ‘%u’ は引数の型が ‘unsigned int*’ であると予期されますが、第 3 引数の型は ‘uint32_t {aka unsigned int}’ です [-Wformat=]
sscanf( src, "%u", net_No );

// srcをネットワーク構造体に変換
sscanf( src, "%u", net_No );

どうしたらよいでしょうか？

[ht3433]

上記のコメントについては、ソースコードを新たにコーディングしたので大丈夫です。