cmcfarlen opened a new pull request, #13132:
URL: https://github.com/apache/trafficserver/pull/13132

   This came up in a copilot review of a fix unrelated to this issue so I 
created this separate PR to address this.
   
   It seems the loop structure at some point looped over an index, but was 
changed to a range based loop, neglecting to change one aspect.  But it was not 
noticed because the index variable is declared before the loop.
   
   It does seem like this is a subtle bug since the index variable could be 
adjusted by the inner loop that increments is, potentially skipping random 
targets in the list.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to