strtolは結果の不一致ながら、文字列変換

Question

私は必要な解決策を見つけていなかったので、私は次のようにstrtolを使用しようとしたthis質問をチェックアウトした後：

：このコードは、いくつかの中間値を生成

in = (unsigned char *)malloc(16); 

for(size_t i = 0; i < (size_t)strlen(argv[2])/2; i+=2) 
{ 
    long tmp = 0; 
    tmp = strtol((const char *)argv[2] + i, &argv[2] + 2, 16); 
    memcpy(&in[i], &tmp, 1); 
} 

全体in配列は0xFF（255）バイトで満たされますと、なぜtmpが等しくない理由を誰かが私に説明していただけますそれは推定値ですか？

inの配列に正しい16進値を入力するように上記のコードを改善する方法についてのヒントも歓迎します。

Answer 1

あなたのコードは、複数のカウントのために誤っているとキャストが問題を隠す：

：

mallocの戻り値はCで必要ではなく、あなたが<stdlib.h>を含めることを忘れた場合、潜在的に危険な変換を非表示にすることができますキャスト(size_t)にstrlenの戻り値をキャスト

in = (unsigned char *)malloc(16); 

は size_t

を返すように定義されていても冗長strlenとして、無駄です。再び、あなたは<string.h>を含めるのを忘れているかもしれません...

for (size_t i = 0; i < (size_t)strlen(argv[2])/2; i += 2) { 
    long tmp = 0; 

strtolはargv[2] + iが暗黙的に変換されますcharへのconstポインタを取ります。キャスト(const char*)は役に立たない。 2番目の引数はchar*のアドレスです。あなたのループの目的は、非常にあいまいですが、あなたはmemcpyでtmpでlong値をコピーする

tmp = strtol((const char *)argv[2] + i, &argv[2] + 2, 16); 

...、あなたがするインデント何を最も確かに、他の用語 &argv[4]で、 argvの5番目の要素のアドレスではなく渡します sizeof(tmp)バイトをコピーする必要があります。 charのサイズとターゲット・システムのエンディアンに応じて、 最初バイトのみの実装が定義されている効果をコピー：

memcpy(&in[i], &tmp, 1); 
} 

はあなたの問題を示し、完全なコンパイルの例を投稿する必要があり、コードの断片でありますどのヘッダファイルが含まれているか、どのように変数が定義されているか、フラグメントの前にどのコードが実行されているかなど、重要なコンテキスト情報がありません。

書かれているように、コードはあまり意味がありません。その動作を解釈しようとするのは無意味です。

参考までに、あなたのコードでは、16進文字列をバイト配列に変換するためのソリューションをリモートで提供していません。