背筋が凍るC言語ソース
関連記事
- 明日も仕事!久しぶりの休日出社キター!!ドロドロドロ 【2024年01月19日(金)】
- 終日,アタフタアタフタ 【2024年01月18日(木)】
- うちは今日から仕事始めらしい.もちろん正月ボケ中 【2024年01月04日(木)】
- 緊急対応で現場に走ってきた 【2023年09月21日(木)】
- 休みボケすぎる(爆 ボゲーとしながら,#POSTMAN でREST-APIを叩きまくっているべ 【2023年08月16日(水)】
某所の謎システム,動かねぇ~な話が来て
ソースコードを添削中
アチコチでヤバ気なメモリ開放処理,リソースリーク多発なコードで,すでに変な汗出まくったけど
これが一番強烈だった
#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
>>康ちゃん さま
少なくともメモリは破壊しちょらん(爆
元の実装がダメすぎるんでまぁ,抵抗感を感じるのは間違いない!