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

Signed-off-by: Spencer Schrock <sschrock@google.com>
ossf · Mar 8, 2023 · 5f13a66 · 5f13a66
1 parent 0c090b3
commit 5f13a66
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