全く分からんです。

この直し方で修正するってことは、他にも「直さないといけない箇所」がたくさん出てきそうで、影響調査とかめんどいです。

oa.OutputW()の呼出し箇所すべてで考慮が必要ですよね？

渡された引数の内容を処理完了までキープしておく責任があるのは呼ばれた側のような気がします。

oa.OutputW()の呼出し箇所すべてで考慮が必要ですよね？

呼び出し箇所で問題が起きそうかどうかの見極めは必要だと思います。

文字列リテラル等の定数値を渡している箇所については特に問題無いと思います。
スタック上の文字列領域もリエントラントなので多分大丈夫だと思います。
LSマクロはstatic記憶領域の変数を使う CLoadString::LoadStringSt の呼び出しになりリエントラントではなく問題が発生しました。

OutputW で Grep したところ他にLSマクロと組み合わせて使用している記述は見当たりませんでした。
なので多分大丈夫な気もしますが、考慮漏れで同様の問題が起きる可能性は否定出来ません。

まぁワニワニパニック大好きなのでその時はまたなんとかすれば良いと思います。変更をする事で不具合がたくさん追加されるとかだとさすがにまずいと思いますが。。

渡された引数の内容を処理完了までキープしておく責任があるのは呼ばれた側のような気がします。

それは一概に言い切れないケースバイケースだと思います。

bool COutputAdapterDefault::OutputW(const WCHAR* pBuf, int size)

こういうプロトタイプのメソッドの呼び出しですが、第1引数のポインタが指す領域の内容が変わる事は、少なくとも利用する呼ぶ側では意識しないと思います（なのでLSマクロのままにしたいんだと言われたら納得しちゃいそう）。呼ばれる側の実装でもなかなか意識しにくいので問題が生じうる実装になっているのではないかと。

影響確認実施済みなのは了解しました。
また、oa.OutputW()の呼ばれた側で対策するのがつらいのもなんとなく分かりました。