背筋が凍るC言語ソース
関連記事
- 働く人の「ストレスチェック」、全事業所に義務拡大へ…昨年度の労災認定は過去最多の883人 【2024年10月11日(金)】
- 那須ハイランドパークの「時給2500円」バイトに応募殺到、10人の枠に100人超が応募 【2024年08月06日(火)】
- 荒っぽい見積書が出来たところでごわす 【2024年08月01日(木)】
- 新入社員4割が転職検討 引き留めに企業が対策 【2024年05月08日(水)】
- 【#C #CPP】謎不具合を追っかけてた.なんとなく原因は分かったけど... 【2024年04月02日(火)】
某所の謎システム,動かねぇ~な話が来て
ソースコードを添削中
アチコチでヤバ気なメモリ開放処理,リソースリーク多発なコードで,すでに変な汗出まくったけど
これが一番強烈だった
#define inf_DataCount 25 int func() { char *timedata[inf_DataCount]; timedata[inf_DataCount] = strtok_s(NULL, ":", &inf_strtok_s_storage); }((((;゚Д゚))))ガクガクブルブル
(判る人だけ判ればいい!:花巻さん爆)
ここの下り char *hogehoge[inf_DataCount];
な配列が10個ほど,関連コードは100行近い
これぞまさしく,バッファオーバーフロー祭!...ドロドロドロ
っちゅうか,配列いらんやろ(爆
文字列処理とポインタ処理がゴッチャになってる
ここいら,全面的に書き直し指示だなこりゃ
C言語はこれがあるから怖いのよのぉ
平気な顔して動き続けてある日突然破裂するべ(笑
書き直し指示ってことは外注に出したソースでしょうか?
ポインタわかってない方にCやらすのが間違いかと(毒爆
ってか、配列要素0開始だったような・・・ポインタ以前の問題!?
>>康ちゃん さま
立場上イロイロ指導したりするわけだわ(謎
指導先で動いている謎システムを添削中
>>配列要素0
C言語は0スタートやじ
>>timedata[inf_DataCount – 1] = strtok_s(NULL, “:”, &inf_strtok_s_storage);
で書いてあればセーフ!(笑
>>timedata[inf_DataCount – 1] = strtok_s(NULL, “:”, &inf_strtok_s_storage);
>で書いてあればセーフ!(笑
セーフというには抵抗あるなぁwww
>>康ちゃん さま
少なくともメモリは破壊しちょらん(爆
元の実装がダメすぎるんでまぁ,抵抗感を感じるのは間違いない!