Fix integer overflow in ecmult_multi_var when n is large
ClosedPublic
Actions

Authored by deadalnix on Jan 17 2020, 12:13.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGINGfd66a562632e: Fix integer overflow in ecmult_multi_var when n is large
rABCfd66a562632e: Fix integer overflow in ecmult_multi_var when n is large

Summary

This is a backport of secp256k1 PR568

Depends on D4985

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

secppr568

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 9010
Build 15982: Default Diff Build & Tests
Build 15981: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jan 17 2020, 12:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 17 2020, 12:13

Fabien accepted this revision.Jan 17 2020, 13:33

This revision is now accepted and ready to land.Jan 17 2020, 13:33

Build Bitcoin-ABC / Diffs / Diff Testing started.

This revision was landed with ongoing or failed builds.Jan 17 2020, 14:09

Closed by commit rABCfd66a562632e: Fix integer overflow in ecmult_multi_var when n is large (authored by Jonas Nick <jonasd.nick@gmail.com>, committed by deadalnix). · Explain Why

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCfd66a562632e: Fix integer overflow in ecmult_multi_var when n is large.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B9010: Diff 15574.Jan 17 2020, 14:19

deadalnix added a commit: rSTAGINGfd66a562632e: Fix integer overflow in ecmult_multi_var when n is large.Apr 7 2020, 22:31

Revision Contents
Changeset List

Path

Size

src/

secp256k1/

src/

ecmult_impl.h

36 lines

tests.c

45 lines

Diff 15574

View Options

src/secp256k1/src/ecmult_impl.h