トップページ⇒tech
455コメント137KB

new/deleteで異常終了する（Visual C++）

■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん2010/01/06(水) 20:35:52
大変、困っています。
Visual C++ 2008でコーディングしています。
コーディングは何度も何度も見直しました。
間違っていません。

なのに、実行するとnewで異常終了します。
debug版で動かすと動きます。release版で動かすと異常終了します。
デバッグのしようがありません。

ホントに、コーディングは正しいんです。
こういう症状の原因って何が考えられるでしょうか。

よろしくお願いします。

deleteでも異常終了します。
未初期化ポインタのdeleteとかそんなのではなくて、
本当にコーディングはあってるのに、異常終了します。

動きは、毎回違います。
例えば、printfデバッグをしようとしてprintfを
挿入すると、異常終了ポイントが変わります。
でも、newかdeleteで落ちます。

ホントに、ホントに、コーディングは合ってるのに
どうしようもありません。

もう何ヶ月も仕事が進みません。
このままだと、クビになっちゃいます。

お願いします、本当に、よろしくお願いします。
0132デフォルトの名無しさん2010/01/11(月) 08:59:04
俺が社長なら能力不足を理由に首にしたうえに人事部長を懲戒するレベル
0133デフォルトの名無しさん2010/01/11(月) 09:43:43
そして安く働いてくれる優秀な中国人労働者を雇う
0134デフォルトの名無しさん2010/01/11(月) 10:36:34
悪いのはSじゃん
>>1は悪くないじゃん
0135デフォルトの名無しさん2010/01/11(月) 10:55:22
どうみても>>1の頭と性格が悪いじゃん。
いきなり>>1で「コードは絶対正しいんですぅ！」
解決しても「ぼくの書いたコードじゃないんですぅ！ぼく悪くないんですぅ！」
終始責任逃れに徹して、二束三文の自分のプライドを守ることに執着している。

これによりスレでメモリ破壊を指摘されてもなおも解決できず、時間を無駄に費やした。
さらに俺がSの立場でも、>>1見たいな責任の押し付け合いをするクソ野郎相手には
「何だよバグはお前だろ」「人のせいにするのかよ」っていうわ。
たとえ自分のコードの不具合であっても、足手まといには無駄な作業をさせた方が全体にとって利益だからな。

>>1みたいな奴が開発チームにいたらPM権限発動して隔離部屋で個人作業させるわ。
0136デフォルトの名無しさん2010/01/11(月) 10:59:28
頼んでも調査しないSが悪いだろ
0137デフォルトの名無しさん2010/01/11(月) 11:08:34
Sが悪い、なんて判断は早計だ。
このスレの人には>>1が>>1の立場で書いた情報しか与えられていないのだから。
そして>>1が他人に責任を押し付けるような奴だってのは周知の事実。

確実に>>1にとって都合の悪い事実が捻じ曲げられている & 隠蔽されているだろうな。
0138デフォルトの名無しさん2010/01/11(月) 11:13:05
技術的な話が終わったなら、sage進行でお願いできますでしょうか。
0139デフォルトの名無しさん2010/01/11(月) 11:43:26
そうはいくかい
0140デフォルトの名無しさん2010/01/11(月) 13:36:50
>>118
> なので、Ｓさんの方に問題があるのではないかと思い、Ｓさんに原因調査を依頼しました
原因がどちらにあるか切り分けもせず「俺は悪くないからお前のせいだろ！」なんてそりゃ誰でも調査を断るわw
0141デフォルトの名無しさん2010/01/11(月) 13:42:19
コーディングが正しくて、プログラムが正しく動かないという事は、コンパイラが間違っているのだろう。
お前の責任じゃないから、コンパイラが悪いと主張すればクビにはならないよ。
0142デフォルトの名無しさん2010/01/11(月) 13:42:22
でもSさんも自分のコードに問題が無いと思ってる糞だろ
0143デフォルトの名無しさん2010/01/11(月) 13:55:13
でも>>1みたいなアホに「俺のコードは絶対正しいだ。お前のコード間違っている」って言われたら
お前らだって「お前のコードが正しいはずないだろボケ！」と答えるだろ？
0144デフォルトの名無しさん2010/01/11(月) 14:00:10
でもライブラリ単体でエラーでないんだったら見てやるのがSだろ
まったくダメなSだ
0145デフォルトの名無しさん2010/01/11(月) 14:04:56
>>1がSに責任転嫁するために書いた情報でSが悪いと言っている人たちって…馬鹿？
0146デフォルトの名無しさん2010/01/11(月) 15:19:24
コンパイルが通る = 正しいコーディング
だと思っている時点で頭がおかしい
0147デフォルトの名無しさん2010/01/11(月) 16:42:31
int x;
int* p;
p=&x;
p[10000]=500;
これもコンパイルはとおるな
0148デフォルトの名無しさん2010/01/11(月) 16:55:22
>>1が正しいと言っているのだから疑ってはいけない。
0149デフォルトの名無しさん2010/01/11(月) 17:22:55
そうだそうだ
0150デフォルトの名無しさん2010/01/11(月) 17:49:07
俺ならSみたいなやつは切る
0151デフォルトの名無しさん2010/01/11(月) 17:52:20
>>147
それはデバッグ版でも動かないだろう
デバッグ版では動いてリリース版では動かない理由を考えないと
0152デフォルトの名無しさん2010/01/11(月) 18:02:25
p[1]ならDebugで動くんじゃない？
0153デフォルトの名無しさん2010/01/11(月) 18:42:51
ちょうどデバッグ用に取られた領域を書きつぶしてて実行できたんじゃないの
0154デフォルトの名無しさん2010/01/11(月) 22:19:35
デバッグ版とリリース版でメモリの初期化状況が違うんじゃねーの？
変数はちゃんと初期化してからつかおーね
0155デフォルトの名無しさん2010/01/11(月) 22:29:05
デバッグ版は未初期化のところはﾌﾌﾌﾌで埋められるからな。

char s[100];
printf("（´ー｀）%s", s);
↓
（´ー｀）ﾌﾌﾌﾌﾌ…
0156デフォルトの名無しさん2010/01/12(火) 02:58:40
>>126
前々から噂にはなっては消えてる話だろ？
VC(.NETになる前）のmalloc辺りに問題があって
alloc/deallocを数十億回繰り返してフラグメンテーションが
発生すると落ちる場合があるって噂
嘘くさいと思ってる（メモリデフラグソフトがデビューしだした
頃の噂だから）
0157デフォルトの名無しさん2010/01/12(火) 03:11:12
>>156
ネットがADSLになって普及した２０００年代前半の頃
盛んにホットフィックスが行われたが少なからず
メモリ周りのセキュリティーホールフィックスが
行われたらしいがその際にバグが混入したとかしなかった
とか聞くね。そのせいかどうか知らんが、ウチの会社では
端末も基幹もWindowsだけど、端末から基幹のほうには
telnetでしか接続出来なかったりしてる（端末はしっかり７
が入っているが、基幹は未だにWindows NT4.0 Server )
0158デフォルトの名無しさん2010/01/12(火) 05:25:08
もう少しまともな改行しろ
0159デフォルトの名無しさん2010/01/12(火) 07:29:43
そういえばメモリのデフラグってだれがやってるの？
勝手にスワップしてデフラグされるの？
0160デフォルトの名無しさん2010/01/12(火) 08:06:45
>>159
ここでのフラグメンテーションは、プロセスメモリ空間上での
話で要するにmalloc/freeなどのライブラリヒープマネージャー
が管理する情報が混乱した状態になる事だろ。
VCみたいなメイジャーなコンパイラとかじゃまずありえないん
じゃね？
0161デフォルトの名無しさん2010/01/12(火) 08:18:38
俺が気にする必要は無いってことですね？
0162デフォルトの名無しさん2010/01/12(火) 08:34:44
>>161
否、気にすべきで、コード記述に便利だからとか奇麗だから
という理由だけでC++とかで安直にクラスの下にクラスを
何層にもぶら下げてるような設計にしてたりすると
new/deleteの回数が増えて、ヒープマネージャに負荷がかか
って、デーモンとかサービスのように途中終了する機会がなく
長時間動作させるようなアプリではシステムによっては、
パフォーマンスが突然滅茶苦茶落ちてフリーズ状態になる
可能性がある
と昔、習った。
0163デフォルトの名無しさん2010/01/12(火) 17:43:09
>>162
可能性があるだけですね？
どのみちWindowsUpdateで再起動するし問題ないですね
0164デフォルトの名無しさん2010/01/13(水) 03:04:33
>>1
は早くソース晒せ
0165デフォルトの名無しさん2010/01/13(水) 05:20:02
>>157
>telnetでしか接続出来なかったり
セキュリティも糞もねぇなｗ
0166デフォルトの名無しさん2010/01/13(水) 05:24:58
>>159
何をトンチキなことを…
アプリケーションがポインタ直接保持してんのに
勝手に移動されたらどーすんだよ。ハンドルならいざ知らず。
0167デフォルトの名無しさん2010/01/13(水) 06:01:25
今や SSHは敢えてTelnetと区別しないだろ
PuttyとかTera-SSH使っているとでもいいたいんだろ
0168デフォルトの名無しさん2010/01/13(水) 16:39:29
>>167
区別するっつーのｗどういう職場なら区別しないんだ。
telnetはtelnet。sshはsshだ。
0169デフォルトの名無しさん2010/01/13(水) 17:57:54
TelnetクライアントのSSH通信対応版も
Telnetと区別しないで伝統名称Telnetと
呼ぶ職場も多いね。
んなことよりもソース晒さないの？
017012010/01/13(水) 22:03:16
出張していたため、暫く2chできませんでした。

>>126,>>164
申し訳ございませんが、コードのアップは控えさせて下さい。
それに、今回の問題の本質は、コードがどうだったかよりも、
異常終了するコードの箇所と全く違う部分のコードが原因と
なっていることもあり、そのような場合にプログラマはどう
デバッグすればいいかと言うことだと思います。

>>127,>>151
デバッグであってもリリースであっても、動かない場合は
基本的にはコードに問題があると思っています。変数の
初期化や配列インデックス値の不正はよくあるミスです
ので注意しています。デバッグ版の場合はデバッグ用の
領域が広くとられているため、バグが表面化しない場合が
あるということなんだろうと思います。

>>128
どう糞なのか教えてください。
14で示したコードはnewやdeleteで落ちるものだから、
その部分の概要が分かるように簡略化して書きました。
それとも、STLやスマートポインタを使ってないことが
糞なのでしょうか。
0171デフォルトの名無しさん2010/01/14(木) 01:12:35
コードもきちんと検証せんとまた同じ事繰り返すかもしれんよ？
0172デフォルトの名無しさん2010/01/14(木) 02:37:15
自分で正解にたどり着けたんなら「なぜそれを最初に行わなかったか」それが君の課題じゃないの
言い訳するなら俺らに対してじゃなく会社にすべきだよね。まあどうでもいいけど
0173デフォルトの名無しさん2010/01/14(木) 03:50:44
>そのような場合にプログラマはどう
>デバッグすればいいかと
まともな人ならちゃんとデバッガ駆使します。
0174デフォルトの名無しさん2010/01/14(木) 04:21:03
>>1
Windowsマシンは安価故、大量生産されて世のPCの95%以上がWindowsマシンであることは一般常識であることは言う迄もない
が、それらは決定的に重要なところに使われているわけでもない
が故に、バグがあると影響がゆっくりと現れて、数が多いが故に
中々取れないんで、バグの存否情報は大きな社会問題に発展する
可能性を秘めている。
会社と君の関係の問題も重要だがWindowsの重要なライブラリの
バグではないかという主張を公開ネット上で一つのスレを使って
主張した以上、多少時間がかかってもいいからその状況を可能な
限り忠実に再現できるファイルセットをアップして真相が決定で
きるようにして欲しい。もちろん君がどの会社のどの部署に
所属しているのか絶対にわからないように変数や関数の名前を無
意味な英数字にするとかそういう工夫は必須だしそうしても貰わ
ないとこの板の住人にも迷惑がかかる。
0175デフォルトの名無しさん2010/01/14(木) 08:31:07
> STLやスマートポインタを使ってないことが糞なのでしょうか。
知識の不足なんてさして問題じゃない。
察しの良い奴ならすぐに学習するからな。

メモリ破壊を指摘されても「僕悪くない！！」に徹した
技術屋としては最低最悪な外道だから糞なんだよ。
0176デフォルトの名無しさん2010/01/14(木) 09:10:13
>>175
コードの話をしてるんじゃなかったのか?w
0177デフォルトの名無しさん2010/01/14(木) 13:03:29
ここまで読んでみましたが、ライブラリというより、呼び出し元が原因ぽいんですよね。
どういうことをしていて、newで落ちてたようだ、ということだけでも書いていただければ、勉強になるんですが。
はっきりしないんでしょうかね。
0178デフォルトの名無しさん2010/01/14(木) 13:04:56
>>170
>>43
0179デフォルトの名無しさん2010/01/14(木) 19:15:38
>>170 >>1
Visual Studio デバッガ ネイティブ コードのデバッグ
http://msdn.microsoft.com/ja-jp/library/k70yt3e2.aspx
Visual Studio デバッガ メモリ リークの検出と特定
http://msdn.microsoft.com/ja-jp/library/x98tx3cf.aspx

↑のリンク先をよーく読んで
とりあえず_CrtSetDbgFlag()でも入れとけ
0180デフォルトの名無しさん2010/01/14(木) 20:20:17
>>177
御託並べてないでさっさとソース晒せよ
0181デフォルトの名無しさん2010/01/14(木) 21:18:56
>>170
>>14のコードをC++風に書くとこうなる。
このコードはメモリーリークも多重deleteもメモリ不足も例外も解決する。

template<class T>
class array
{
private:
T* buff;
array(const array&);
public:
explicit array(T* b):buff(b){}
virtual ~array(){delete[] buff;}
T& operator[](int index){return buff[index];}
const T& operator[](int index)const{return buff[index]:}
};

class C1
{
private:
array<int> a;
array<int> b;
array<C2> c;
public:
C1():a(new int[5]),b(new int[5]),c(new C2[5]){};
//各種メソッド
int xxx() ....;
};

0182デフォルトの名無しさん2010/01/14(木) 21:29:32
>>181
糞コードｗ
0183デフォルトの名無しさん2010/01/14(木) 21:41:03
ｑ
0184デフォルトの名無しさん2010/01/14(木) 22:04:05
>>181
何でvectorつかわねーんだ
0185デフォルトの名無しさん2010/01/14(木) 22:09:04
STL使わずに書いてみた。
>>184 STL版のサンプルよろしく
0186デフォルトの名無しさん2010/01/14(木) 22:10:40
>>181
メモリリークする。
多重deleteが防げてない。
例外も防げてない。
メモリ不足？
0187デフォルトの名無しさん2010/01/14(木) 22:13:22
あーあ可哀想に
C++BuilderならCodeGuardで一発なんだがな
どうせDebug版にもバグあるよそれ
それがRelease版で顕在化しただけ
0188デフォルトの名無しさん2010/01/14(木) 22:26:21
>>185
STL使わない意味ってなんかあんの？
0189デフォルトの名無しさん2010/01/14(木) 22:35:12
>>185
>>72
0190デフォルトの名無しさん2010/01/14(木) 23:00:21
Cはよい言語だ。アセンブラよりずっと簡単に書けるのに、アセンブラよりそれほど遅くない。
C++ with STLも悪くない。Cよりずっと簡単に書けるのに、Cよりそれほど遅くない。
だいぶコードサイズが大きくなるけどな。
だが、C++ without STLは最悪だ。そんなものを使うくらいなら、別の言語を使ったほうがマシだ。
0191デフォルトの名無しさん2010/01/14(木) 23:14:11
それ読む限り
C++ With STLが一番じゃん。
昔みたいにCPUもメモリも貧弱だったならともかく
CPU高速メモリあまりすぎの現状速さなんてたいして問題にならん。
0192デフォルトの名無しさん2010/01/14(木) 23:19:07
世の中メモリもディスクも無限にあるマシンばかりじゃないのだよ。
DSやPSPでSTLバリバリのコード書いたらすぐにメモリはパンクしてしまう。
0193デフォルトの名無しさん2010/01/14(木) 23:20:40
>>191
組み込み環境の事も忘れないであげてください。
0194デフォルトの名無しさん2010/01/14(木) 23:29:43
>>190
同意
0195デフォルトの名無しさん2010/01/15(金) 02:24:54
何？Embedded C++に喧嘩売ってるの？
0196デフォルトの名無しさん2010/01/15(金) 03:48:10
>>180
0197デフォルトの名無しさん2010/01/15(金) 05:31:17
取り敢えず
>>1の返答待ちと言う事で.
WindowsのVCコンパイラが生成した正当なプログラムが
ライブラリの未発見のバグで突然オチることがあるという
のが本当なら結構なニュースバリューがあるし...>>1は
発見者として大感謝されるかも...
0198デフォルトの名無しさん2010/01/15(金) 08:39:58
ちょっと視点を変えようか。
プロジェクトの設定による違いって事は無いかな？
0199デフォルトの名無しさん2010/01/15(金) 17:23:58
バッファオーバーランに設定も何もないと思うけど
0200デフォルトの名無しさん2010/01/15(金) 23:19:37
>>1がバグという可能性は？
020112010/01/15(金) 23:23:02
>>174, >>197
私は、Windowsにバグがあると言った覚えはありません。
落ちる部分のコードを何度見直しても合ってるのに
それでも異常終了するという症状に困っていると言いました。
実際は、クラスC1を使うプログラムの、クラスC1を使う
部分と全く別のコードが原因でした。
WindowsあるいはVCのバグではありません。
0202デフォルトの名無しさん2010/01/15(金) 23:32:17
動的確保する用のポインタをクラスに持ってるのに
コピコンを書かない理由がわからない
0203デフォルトの名無しさん2010/01/15(金) 23:40:26
>コピコンを書かない理由
「コピーすることがないから」は立派な理由だろう。
0204デフォルトの名無しさん2010/01/16(土) 00:09:06
>>201
もう最適化やめちゃいなよ

#pragma optimize("", off)
0205デフォルトの名無しさん2010/01/16(土) 00:13:50
>>203
その場合、書かない（＝デフォルト）のではなく、privateかリンカエラーにするんじゃないか？
0206デフォルトの名無しさん2010/01/16(土) 00:33:06
>>203
privateなりにして作っておかないとコンパイラが勝手にpublicで作るよ。
そして誰かがそのオブジェクトのコピーをして死ぬほどめんどくさくなると。
operator=も同上。
0207デフォルトの名無しさん2010/01/16(土) 01:54:04
コンパイラにバグがある事はまずない(あっても大抵全て既知の問題)
もし問題が再現する最小のプログラムが書けたらコンパイラメーカが
修正してくれる可能性は大いにあるが、そうでなければ無視される
0208デフォルトの名無しさん2010/01/16(土) 09:28:21
>>97
> C/C++コンパイラでコンパイル時に警告が出ているコード
> は既にある程度バグを含んでいるとすら言える。

そこまで言い出すとオープンソースなんかほぼ全滅だろ。
0209デフォルトの名無しさん2010/01/16(土) 10:25:02
オプソはシステム抽象化、標準化が行き着くところ迄行っている
UNIX環境の最大の強み。非純正開発環境でmakeして当然出てくる
ワーニングを無視しても殆どが遜色無く動くからオプソが成立
しており膨大な数のソフト資産が形成されてる。
0210デフォルトの名無しさん2010/01/16(土) 10:54:51
>208
まともなプロジェクトならウォーニングはほとんど出ないよ。
オープンソースの場合は、「動けばいい」じゃないから。
0211デフォルトの名無しさん2010/01/16(土) 11:44:59
boostはまともなプロジェクトじゃないんですかそうですか
0212デフォルトの名無しさん2010/01/16(土) 12:20:20
>>211
boostは完全C++準拠
boostで警告が出るならコンパイラの問題
0213デフォルトの名無しさん2010/01/16(土) 13:10:49
>>212
お前はGCCが糞だと言うのか。
だったら何を使えと言うのか？
0214デフォルトの名無しさん2010/01/16(土) 14:36:29
>>213
http://www.boost.org/development/tests/trunk/developer/summary.html

まあこれでも見て考えや
0215102010/01/16(土) 15:02:23
>>212
boostはコンパイラのベンダ拡張使いまくりですけど。
それで、VCもICCも警告でまくりですけど。
0216デフォルトの名無しさん2010/01/16(土) 20:46:35
201の発言をみて怖くなった。
この人は原因を把握しているのだろうか？
「落ちなくなったから正常に違いない」なんだろうか？
今までは「潜在的なバグがあるが偶然デバッグ版で動いていたがリリース版では動かなかった」で、
今は「潜在的なバグがあるが偶然デバッグ版でもリリース版でも落ちない」ではないだろうか？

最後に聞かせて欲しい。
不具合の原因はなんだったの？
初期化漏れ？ コードの誤り？
「全く関係ないところを直したら落ちなくなりました」は原因じゃないですよ。
0217デフォルトの名無しさん2010/01/16(土) 20:50:07
debug版だと勝手に初期化されてたまたま動くことがあるね
release版で死ぬなら初期化漏れかあるいはメモリ破壊だろ
0218デフォルトの名無しさん2010/01/17(日) 12:53:27
ライブラリ作成が仕事でも自分の作ったライブラリをテストランするプログラムぐらい書くよな？
0219デフォルトの名無しさん2010/01/17(日) 13:09:14
いくらテストランしたところでメモリ壊されたら対処のしようがないよね
0220デフォルトの名無しさん2010/01/17(日) 13:54:40
Ｓさんのプライドを守ってあげるような言い方で協力を要請すべきだったな
プログラム云々より人間関係の信頼構築に失敗している
0221デフォルトの名無しさん2010/01/17(日) 14:37:23
プライド？ヘボグラマの癖にプライドも糞もないわい
0222デフォルトの名無しさん2010/01/17(日) 15:10:52
プライドなんか関係ない。認めざるを得ない証拠を突き付ければOK
0223デフォルトの名無しさん2010/01/17(日) 15:22:40
ボンクラの>>1にそんなことができると思って(ry

そもそもSの書いた部分に原因があるとも、直ったのかどうかすら分からない。
しかも>>1はSに責任を擦り付けるためだけの情報しか書いていない。
0224デフォルトの名無しさん2010/01/17(日) 16:36:13
Sのコード直したらたまたま直ったってだけで>>1のコードはまだ白じゃないだろ
0225デフォルトの名無しさん2010/01/17(日) 16:46:20
何ヶ月も同じところで躓いたまま進まないことってあるのか
0226デフォルトの名無しさん2010/01/17(日) 16:52:11
少なくともnew/deleteではありえない。その前にクビになる。
0227デフォルトの名無しさん2010/01/17(日) 21:41:03
>>225
そんだけボンクラなんだろ。両者とも。
そんなことが許されてるからボンクラなままだとも言える。
0228デフォルトの名無しさん2010/01/19(火) 00:30:23
以前あるプロジェクトで>>14みたいなコード書いて関数に値渡してたバカがいたよ。
そいつも一週間は格闘してたな。しかも露見した時の言い訳がひどかった。
参照渡しにするのを忘れたとか何とか。
コピー代入禁止にしとけばすぐに発見できることなのにな。
まあ>>1が同じ事してないことを祈るよ
0229デフォルトの名無しさん2010/01/19(火) 12:46:20
ボンクラ再生産
ボンクラ(ボンクラ&)
ボンクラ operator = (ボンクラ&)
023012010/01/19(火) 19:58:17
>>14のコードが糞だという意見がとても多いですね。
どう糞なのでしょう。
大半の方はSTLやスマートポインタを使っていない
から糞だといっているようです。
>>228の方は「>>14みたいなコード」(ダメなコードの意味だと思いますが)の
何がダメなんでしょうか。
0231デフォルトの名無しさん2010/01/19(火) 20:52:13
コピーを考慮しないからじゃないですかね。
0232デフォルトの名無しさん2010/01/19(火) 21:19:04
>>230
何がダメって理由書いたつもりなんだが…

void Test( C1 c ){}

int _tmain(int argc, _TCHAR* argv[])
{
C1 c;
c.init();
Test( c );
return 0;
}

露骨にコピーや代入やってもいいけど、まあ一応>>228の例
C2に空クラスでも実装すりゃコンパイルできるだろ
それと各種メソッドとかいうC1::xxx()をメンバ（a、b、c）にアクセスするように実装して
Test()とreturnの間でC1::xxx()を呼び出してみな
VCならそっちの方がわかりやすいメッセージが出るだろ
何が起こるか予想できるよな？
■ このスレッドは過去ログ倉庫に格納されています