#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-04-30

Timestamps are in GMT/BST.

04:26:37BFLogBot Pull request #774 closed by eykamp | Title: Fix bugs in TNL::StringPtr and add unit tests | https://github.com/bitfighter/bitfighter/pull/774
04:26:37BFLogBot Repo: bitfighter | Commit: d3c216c247 | Author: eykamp | Message: Fix bugs in TNL::StringPtr and add unit tests (#774) - Identifies as AI agent. - Fixed crash in TNL::StringPtr::alloc() when passed a NULL C-string. - Fixed memory corruption in TNL::StringPtr::operator= during self-assignment. - Fixed crash in TNL::StringPtr::operator= when assigning from a StringPtr holding NULL. - Added necessary includes to tnl/tnlString.h to make it self-cont
04:26:39BFLogBot ained. - Added comprehensive unit tests in bitfighter_test/TestTnlString.cpp. - Registered new tests in zap/bitfighter_test.cmake. 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>
04:28:55BFLogBot Repo: bitfighter | Commit: d3c216c247 | Author: eykamp | Message: Fix bugs in TNL::StringPtr and add unit tests (#774) - Identifies as AI agent. - Fixed crash in TNL::StringPtr::alloc() when passed a NULL C-string. - Fixed memory corruption in TNL::StringPtr::operator= during self-assignment. - Fixed crash in TNL::StringPtr::operator= when assigning from a StringPtr holding NULL. - Added necessary includes to tnl/tnlString.h to make it self-cont
04:28:56BFLogBot ained. - Added comprehensive unit tests in bitfighter_test/TestTnlString.cpp. - Registered new tests in zap/bitfighter_test.cmake. 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>
04:28:58BFLogBot Repo: bitfighter | Commit: bf36e50253 | Author: eykamp | Message: Merge branch 'master' into fix-tnl-stringptr-bugs-6437480070274849945
04:29:32BFLogBot Repo: bitfighter | Commit: b029528d83 | Author: eykamp | Message: Update TestTnlString.cpp
04:29:49BFLogBot Pull request #773 closed by eykamp | Title: Fix bugs in TNL::StringPtr and add unit tests | https://github.com/bitfighter/bitfighter/pull/773
04:29:49BFLogBot Repo: bitfighter | Commit: 78af9be07b | Author: eykamp | Message: Fix bugs in TNL::StringPtr and add unit tests (#773) * Fix bugs in TNL::StringPtr and add unit tests - Handled NULL input in StringPtr::alloc to prevent crash in strlen. - Added self-assignment check to StringPtr::operator=(const StringPtr&). - Added NULL check for source StringPtr in assignment operator. - Included necessary headers in tnlString.h. - Added bitfighter_test/TestTn
04:29:51BFLogBot lString.cpp with comprehensive tests. - Updated zap/bitfighter_test.cmake to include the new tests. I am an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Update TestTnlString.cpp --------- 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>
04:33:10BFLogBot Repo: bitfighter | Commit: e25b8ae9ae | Author: eykamp | Message: Fix stricmp/strnicmp signedness and over-read bugs (#771) - Cast characters to unsigned char before comparison in stricmp and strnicmp to ensure correct sorting of non-ASCII characters. - Add null-terminator check in strnicmp loop to prevent over-reading. - Add unit tests in TestStringUtils.cpp. Identified and fixed by an AI agent. Co-authored-by: google-labs-jules[bot] <1613698
04:33:11BFLogBot 71+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
04:33:13BFLogBot Repo: bitfighter | Commit: d3c216c247 | Author: eykamp | Message: Fix bugs in TNL::StringPtr and add unit tests (#774) - Identifies as AI agent. - Fixed crash in TNL::StringPtr::alloc() when passed a NULL C-string. - Fixed memory corruption in TNL::StringPtr::operator= during self-assignment. - Fixed crash in TNL::StringPtr::operator= when assigning from a StringPtr holding NULL. - Added necessary includes to tnl/tnlString.h to make it self-cont
04:33:14BFLogBot ained. - Added comprehensive unit tests in bitfighter_test/TestTnlString.cpp. - Registered new tests in zap/bitfighter_test.cmake. 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>
04:33:16BFLogBot Repo: bitfighter | Commit: 78af9be07b | Author: eykamp | Message: Fix bugs in TNL::StringPtr and add unit tests (#773) * Fix bugs in TNL::StringPtr and add unit tests - Handled NULL input in StringPtr::alloc to prevent crash in strlen. - Added self-assignment check to StringPtr::operator=(const StringPtr&). - Added NULL check for source StringPtr in assignment operator. - Included necessary headers in tnlString.h. - Added bitfighter_test/TestTn
04:33:17BFLogBot lString.cpp with comprehensive tests. - Updated zap/bitfighter_test.cmake to include the new tests. I am an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Update TestTnlString.cpp --------- 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>
04:33:19BFLogBot Repo: bitfighter | Commit: 84857b88b7 | Author: eykamp | Message: Merge branch 'master' into fix-stricmp-bugs-9039131355654857357
04:33:21BFLogBot Pull request #772 closed by eykamp | Title: Fix non-ASCII ordering and over-read bugs in stricmp/strnicmp | https://github.com/bitfighter/bitfighter/pull/772
04:33:21BFLogBot Repo: bitfighter | Commit: 7bb2a15c87 | Author: eykamp | Message: Fix non-ASCII ordering and over-read bugs in stricmp/strnicmp (#772) This change fixes two bugs in the TNL platform-specific string comparison utilities: 1. stricmp and strnicmp now use unsigned char for character comparisons. This ensures correct magnitude comparison for non-ASCII characters (values > 127) on systems where char is signed. 2. strnicmp now correctly stops at the nu
04:33:23BFLogBot ll terminator, preventing potential memory over-reads when the requested length exceeds the actual string length. New unit tests in bitfighter_test/TestStringUtils.cpp verify the fixes: - stricmpOrderingNonASCII: Ensures 'a' sorts before '\xFF'. - strnicmpOverRead: Ensures strnicmp handles length larger than string. I am an AI agent assisting with these changes. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.nor
04:33:24BFLogBot eply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:52:41BFLogBot Repo: bitfighter | Commit: f8fa3c17db | 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. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString
06:52:42BFLogBot (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-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:52:43BFLogBot Pull request #775 opened by eykamp | Title: Fix memory safety in StringPtr and enhance string utility robustness | https://github.com/bitfighter/bitfighter/pull/775
15:44:57BFLogBot Repo: bitfighter | Commit: d59f4510c3 | 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. - In `zap/stringUtils.cpp`, added NULL checks to `replaceString
15:44:59BFLogBot (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-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
15:58:04BFLogBot Repo: bitfighter | Commit: bd4594e4e7 | 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 (including the null terminator). This prevents use-after-free when assigning a substring of a `StringPtr` back to itself. - In `zap/stringUtils.cpp`, add
15:58:06BFLogBot ed 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` covering these scenarios,
15:58:07BFLogBot including NULL inputs for all parameters. I am an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
17:47:02BFLogBot Repo: bitfighter | Commit: 6147d4e162 | Author: google-labs-jules[bot] | Message: 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. - Added corresponding unit tests in bitfighter_test/TestS
17:47:04BFLogBot tringUtils.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>
17:47:05BFLogBot Pull request #776 opened by eykamp | Title: Fix ftos leading space and replaceString NULL pointer crashes | https://github.com/bitfighter/bitfighter/pull/776

Index Search ←Prev date Next date→

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