お世話になります。
Wikiペディアに掲載されていたstrcatの実装例(http://ja.wikipedia.org/wiki/Strcat#.E5.AE.9F.E8 …)をそのまま使って、
mystrcatを作ってみましたが、結果が空白になってしまいます。
[ソース]
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
char *mystrcat(char *s1, const char *s2)
{
char *p = s1;
while(*s1++);
while(*s1++ = *s2++);
return p;
}
void main(){
char *dst;
char *p;;
char *src1 = "Hello";
char *src2 = "World";
dst = malloc(strlen(src1)+strlen(src2)+1);
printf("%s\n",mystrcat(dst,src1));
printf("%s\n",mystrcat(dst,src2));
free(dst);
}
[実行結果]
空白2行
[期待していた動作]
Hello
HelloWorld
何が間違っているのか教えていただけると助かります。
標準のstrcatを使うとHelloとHelloWorldが表示されます。
よろしくお願いします。
No.5ベストアンサー
- 回答日時:
>ところでmallocの間違いを指摘されていますが、
>どのように問題となるのでしょうか。
>心当たりがあるといえば、\0がないので、
>ゴミを表示するのではないか、と懸念されますが、
その前にたぶん吹っ飛びます。
確保した領域に'\0'がないと、
while(*s1) s1++;
で見つかるまでポインタをインクリメントし続けます。
バッファオーバーランって聞いたことありませんかね?
で、はみ出した場所に書き込もうとして破壊します。
# OSが止めてくれるかそのまま動作してさらに悲惨な状況になるか……。
>memsetでdstを'\0'で埋めれば良いということでしょうか。
それでもいいですし、確保した最初の場所だけに0x00を入れる。でもOKでしょう。
# 私ならmemset()でクリアする。(というかcalloc()使う)
No.4
- 回答日時:
あ, 確かに>#3.
malloc では 0クリアしません>#2. calloc はするけど.
あと, たぶん未定義動作のはずだから「strcat()でも同様に失敗するハズ」ともいえないのがつらい.
No.3
- 回答日時:
while(*s1++);
この行が間違い。
それ以前に、malloc() で確保したメモリ領域には、何が入っているかわからないので、それも問題だけど。
たとえば、上の行を
while(*s1) s1++;
とするだけで、とりあえず、動く。
どのように動きが違うのかを考えて見えるといいと思う。
この回答への補足
while(*s1++ = *s2++);
も間違いで、
while(*s2) *s1++ = *s2++;
が正しいということで理解し修正した結果
期待したとおりの結果となりました。
ところでmallocの間違いを指摘されていますが、
どのように問題となるのでしょうか。
心当たりがあるといえば、\0がないので、
ゴミを表示するのではないか、と懸念されますが、
memsetでdstを'\0'で埋めれば良いということでしょうか。
ついでに教えていただけるとありがたいです。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
関連するカテゴリからQ&Aを探す
おすすめ情報
- ・漫画をレンタルでお得に読める!
- ・街中で見かけて「グッときた人」の思い出
- ・「一気に最後まで読んだ」本、教えて下さい!
- ・幼稚園時代「何組」でしたか?
- ・激凹みから立ち直る方法
- ・1つだけ過去を変えられるとしたら?
- ・【あるあるbot連動企画】あるあるbotに投稿したけど採用されなかったあるある募集
- ・【あるあるbot連動企画】フォロワー20万人のアカウントであなたのあるあるを披露してみませんか?
- ・映画のエンドロール観る派?観ない派?
- ・海外旅行から帰ってきたら、まず何を食べる?
- ・誕生日にもらった意外なもの
- ・天使と悪魔選手権
- ・ちょっと先の未来クイズ第2問
- ・【大喜利】【投稿~9/7】 ロボットの住む世界で流行ってる罰ゲームとは?
- ・推しミネラルウォーターはありますか?
- ・都道府県穴埋めゲーム
- ・この人頭いいなと思ったエピソード
- ・準・究極の選択
- ・ゆるやかでぃべーと タイムマシンを破壊すべきか。
- ・歩いた自慢大会
- ・許せない心理テスト
- ・字面がカッコいい英単語
- ・これ何て呼びますか Part2
- ・人生で一番思い出に残ってる靴
- ・ゆるやかでぃべーと すべての高校生はアルバイトをするべきだ。
- ・初めて自分の家と他人の家が違う、と意識した時
- ・単二電池
- ・チョコミントアイス
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
DLLのマルチスレッドの動作につ...
-
void*型のデータサイズ
-
デストラクタを呼びたい
-
newしないオブジェクトについて
-
C++で、メンバもヒープに確保さ...
-
ヒープメモリの解放について
-
スタック破壊の上手な見つけ方...
-
LPWSTRのコピー
-
allocってなんですか?
-
new と malloc によるメモリの...
-
プログラムが途中で強制終了し...
-
LoadLibraryでAccess Violation...
-
構造体でchar name[]と*nameの...
-
HEAP に関すること
-
メモリ解放について
-
メモリ不足になってしまう。
-
mallocについて
-
指定したメモリアドレスの値の...
-
グローバル変数のサイズ
-
VB.netでのwin32api呼び出し
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
allocってなんですか?
-
newしないオブジェクトについて
-
c言語のポインタへの文字列入力...
-
ヒープメモリの解放について
-
配列の添え字の最大数とは?
-
stringの最大サイズ
-
C++で、メンバもヒープに確保さ...
-
プログラムが途中で強制終了し...
-
void*型のデータサイズ
-
malloc呼び出し時のセグメンテ...
-
スタック破壊の上手な見つけ方...
-
ビットをローテートするプログ...
-
構造体でchar name[]と*nameの...
-
mallocについて
-
GDI+におけるメモリの開放について
-
ポインタのポインタの初期化法
-
構造体を使ったファイルの読み込み
-
C言語 mallocとfreeについて
-
HEAP に関すること
-
VBからMFC-DLL呼び出し
おすすめ情報