algorand / algorand/go-algorand

check alignment of all atomic vars

オープン
#5,001 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
new-feature-request Team Carbon-11
主要言語
Go
スター
1.4k
フォーク
537
平均マージ
1日 7時間
マージ済み PR(30日)
18

説明

## Problem

On 32 bit platforms (ARM) `sync/atomic` operations can crash if a 64 bit value is misaligned.

## Solution

Create unit tests that use `unsafe.Offset` to check if that `% 8 == 0`

See `network/wsPeer_test.go`

## Dependencies

None

## Urgency

Nice-to-have. 32 bit platforms aren't much used.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。