Re: Possible buffer overflow in deprecated wxContractPath() (#15301)

17 views
Skip to first unread message

Richard Thomson

unread,
Jul 11, 2026, 2:33:30 PM (10 days ago) Jul 11
to wx-...@googlegroups.com, wxtrac, Author
LegalizeAdulthood left a comment (wxWidgets/wxWidgets#15301)

I checked the current tree and this appears to have been addressed already.

The reported code path was the deprecated wxContractPath() implementation in src/common/filefn.cpp, using the shared wxFileFunctionsBuffer. That whole v2.8 compatibility block was removed in commit 5e2da5a1066332c86f6c31223e9733337b438163 ("Get rid of v2.8 code"), including the wxContractPath() declaration in include/wx/filefn.h and its implementation in src/common/filefn.cpp.

Current checks:

  • rg wxContractPath finds only the old changelog mention.
  • [filefn] tests pass.
  • wxFileName::Normalize and wxFileName::Replace tests pass.

So the buffer-overflow warning no longer applies to the current code.


Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you authored the thread.Message ID: <wxWidgets/wxWidgets/issues/15301/4948304809@github.com>

VZ

unread,
Jul 19, 2026, 2:17:49 PM (2 days ago) Jul 19
to wx-...@googlegroups.com, wxtrac, Author

Closed #15301 as completed.


Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you authored the thread.Message ID: <wxWidgets/wxWidgets/issue/15301/issue_event/28184938500@github.com>

Reply all
Reply to author
Forward
0 new messages