Atomically load from accessState to avoid data race. (ossf#2732)

Signed-off-by: Spencer Schrock <sschrock@google.com> Signed-off-by: Shofiya2003 <shofiyabootwala@gmail.com>
Shofiya2003 · Mar 10, 2023 · ff2f6a7 · ff2f6a7
1 parent 4c395ae
commit ff2f6a7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/clients/githubrepo/roundtripper/tokens/round_robin.go b/clients/githubrepo/roundtripper/tokens/round_robin.go
@@ -36,7 +36,7 @@ func (tokens *roundRobinAccessor) Next() (uint64, string) {
 
 	// If selected accessToken is unavailable, wait.
 	for !atomic.CompareAndSwapInt64(&tokens.accessState[index], 0, time.Now().Unix()) {
-		currVal := tokens.accessState[index]
+		currVal := atomic.LoadInt64(&tokens.accessState[index])
 		expired := time.Now().After(time.Unix(currVal, 0).Add(expiryTimeInSec * time.Second))
 		if !expired {
 			continue