#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-05-01

Timestamps are in GMT/BST.

02:31:35BFLogBot Repo: bitfighter | Commit: e8f2602718 | Author: google-labs-jules[bot] | Message: Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. Added explanatory comments for the memory management logic. - I


02:31:37BFLogBot n `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ...)` to return an empty string if the input is NULL, return the input if the search string is NULL/empty, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp


02:31:38BFLogBot ` covering these scenarios, including NULL inputs for all parameters and substring self-assignment.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


02:43:32BFLogBot Pull request #775 closed by eykamp | Title: Fix memory safety in StringPtr and enhance string utility robustness | https://github.com/bitfighter/bitfighter/pull/775


02:43:32BFLogBot Repo: bitfighter | Commit: 574a1ede01 | Author: eykamp | Message: Fix memory safety in StringPtr and enhance string utility robustness (#775)  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself.


02:43:33BFLogBot - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ...)` to return an empty string if the input is NULL, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios.  I am an AI agent.  Co


02:43:35BFLogBot -authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ..


02:43:36BFLogBot .)` to return an empty string if the input is NULL, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  * Fix use-after


02:43:38BFLogBot -free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer (including the null terminator). This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ...)` to return an empty string if the input is N


02:43:39BFLogBot ULL, return the input if the search string is NULL/empty, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios, including NULL inputs for all parameters.  I am an AI agent.  Co-authored-by: eykamp <1133323+e


02:43:41BFLogBot ykamp@users.noreply.github.com>  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. Added explanatory comments for the memory management logic. - In `zap/stringUtils.cpp`, added NULL checks to `


02:43:42BFLogBot replaceString(const char*, ...)` to return an empty string if the input is NULL, return the input if the search string is NULL/empty, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios, including NULL inpu


02:43:44BFLogBot ts for all parameters and substring self-assignment.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  ---------  Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


02:45:42BFLogBot Repo: bitfighter | Commit: 574a1ede01 | Author: eykamp | Message: Fix memory safety in StringPtr and enhance string utility robustness (#775)  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself.


02:45:43BFLogBot - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ...)` to return an empty string if the input is NULL, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios.  I am an AI agent.  Co


02:45:45BFLogBot -authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ..


02:45:46BFLogBot .)` to return an empty string if the input is NULL, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  * Fix use-after


02:45:48BFLogBot -free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer (including the null terminator). This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString(const char*, ...)` to return an empty string if the input is N


02:45:49BFLogBot ULL, return the input if the search string is NULL/empty, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios, including NULL inputs for all parameters.  I am an AI agent.  Co-authored-by: eykamp <1133323+e


02:45:51BFLogBot ykamp@users.noreply.github.com>  * Fix use-after-free in StringPtr and enhance replaceString robustness  - In `tnl/tnlString.h`, fixed `StringPtr::operator=(const char*)` to handle aliasing where the input pointer points into its own internal buffer. This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. Added explanatory comments for the memory management logic. - In `zap/stringUtils.cpp`, added NULL checks to `


02:45:52BFLogBot replaceString(const char*, ...)` to return an empty string if the input is NULL, return the input if the search string is NULL/empty, and treat a NULL replacement as an empty string. - In `zap/stringUtils.h`, added the explicit declaration for the `const char*` overload of `replaceString`. - Added comprehensive unit tests in `bitfighter_test/TestTnlString.cpp` and `bitfighter_test/TestStringUtils.cpp` covering these scenarios, including NULL inpu


02:45:54BFLogBot ts for all parameters and substring self-assignment.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  ---------  Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


02:45:55BFLogBot Repo: bitfighter | Commit: 7e6dace3c5 | Author: eykamp | Message: Merge branch 'master' into fix-stringutils-bugs-13864769786456439057


02:47:02BFLogBot Repo: bitfighter | Commit: ea2b494644 | Author: eykamp | Message: Fix merge problems  Add test case for replaceString with NULL input.


02:47:17BFLogBot Pull request #776 closed by eykamp | Title: Fix ftos leading space and replaceString NULL pointer crashes | https://github.com/bitfighter/bitfighter/pull/776


02:47:17BFLogBot Repo: bitfighter | Commit: fa0bcb6a5a | Author: eykamp | Message: Fix ftos leading space and replaceString NULL pointer crashes (#776)  * Fix ftos leading space and replaceString NULL pointer crashes  - In ftos, changed the format string from "%2." to "%." to prevent unwanted leading spaces for single-digit results. - In replaceString, added NULL pointer checks for the input and replacement strings to prevent crashes and ensure robust behavior. -


02:47:19BFLogBot Added corresponding unit tests in bitfighter_test/TestStringUtils.cpp to verify the fixes and prevent regressions.  I am an AI agent assisting with this task.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>  * Fix merge problems  Add test case for replaceString with NULL input.  ---------  Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@us


02:47:20BFLogBot ers.noreply.github.com>


07:33:31BFLogBot Repo: bitfighter | Commit: 2502478171 | Author: google-labs-jules[bot] | Message: jules: fix s_fprintf and readFile bugs in stringUtils.cpp  - Direct file output in s_fprintf to avoid 2048-char truncation. - Precise UTF-8 BOM check in readFile instead of character-set trimming. - Added regression tests for both issues in TestStringUtils.cpp.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


07:33:33BFLogBot Pull request #777 opened by eykamp | Title: Fix s_fprintf truncation and readFile BOM trimming bugs | https://github.com/bitfighter/bitfighter/pull/777


15:15:36BFLogBot Repo: bitfighter | Commit: 1e3ffc681d | Author: eykamp | Message: jules: fix s_fprintf and readFile bugs in stringUtils.cpp (#777)  - Direct file output in s_fprintf to avoid 2048-char truncation. - Precise UTF-8 BOM check in readFile instead of character-set trimming. - Added regression tests for both issues in TestStringUtils.cpp.  Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by:


15:15:36BFLogBot Pull request #777 closed by eykamp | Title: Fix s_fprintf truncation and readFile BOM trimming bugs | https://github.com/bitfighter/bitfighter/pull/777


15:15:37BFLogBot eykamp <1133323+eykamp@users.noreply.github.com>


18:18:31BFLogBot Repo: bitfighter | Commit: 7ad19f13db | Author: google-labs-jules[bot] | Message: Fix TNL::Vector storage and pointer arithmetic  Change the underlying storage of VectorBase from std::vector to std::vector. This ensures that each element occupies exactly one byte, matching sizeof(bool) and allowing correct pointer arithmetic when using the pointer returned by the address() method.  Also add a unit test to verify Vector


18:18:32BFLogBot indexing and address safety.  I am an AI agent.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


18:18:34BFLogBot Pull request #778 opened by eykamp | Title: Fix TNL::Vector storage and pointer arithmetic | https://github.com/bitfighter/bitfighter/pull/778


18:45:10BFLogBot Repo: bitfighter | Commit: d61b0690a4 | Author: google-labs-jules[bot] | Message: Fix TNL::Vector storage and CI workflows  - Change TNL::Vector storage from std::vector to std::vector to fix pointer arithmetic and memory alignment issues in address() and operator[]. - Add unit test bitfighter_test/TestTnlVectorBool.cpp to verify fix. - Fix CI workflows by adding 'mkdir -p build' and 'apt-get update'.  I am an AI agent.  Co-a


18:45:12BFLogBot uthored-by: eykamp <1133323+eykamp@users.noreply.github.com>

Index Search ←Prev date Next date→

These logs were automatically created by BFLogBot on irc.freenode.net.