rust-paillier

rust-paillier copied to clipboard

I noticed that the crate redefines serialization of `curv::BigInt`s to be in base 10, e.g., for `KeyPair`, `p` and `q` are going to be in base 10 https://github.com/ZenGo-X/rust-paillier/blob/7d4958fb73cdae283906449e71f0dbb10ff459b3/src/lib.rs#L23-L29 because https://github.com/ZenGo-X/rust-paillier/blob/7d4958fb73cdae283906449e71f0dbb10ff459b3/src/serialize.rs#L9-L11...

vnermolaev

To increase discoverability on GitHub it could be good to migrate the repository to a standalone repository so that it will pop up in GitHub searches. This seems to be...

zicklag

BigInt type problem (curv and rust-paillier)

2

Hi, I encountered a type conversion issue among type `BigInt`: 1. Using `Paillier::encrypt` in rust-paillier, we need to first convert the `BigInt` variable to `RawPlaintext`. However, there is an error:...

marvellous987

fix links to paillier paper

1

https://github.com/ZenGo-X/rust-paillier/blob/master/src/core.rs#L334

omershlo

Missing checks on membership of r in multiplicative group

1

This issue refers to https://github.com/ZenGo-X/rust-paillier/blob/57fc04d28ac58bfde387305e44715aa95cc3033d/src/core.rs#L325 Ideally, the `r` chosen here should be coprime to `N` in [general Paillier scheme](https://en.wikipedia.org/wiki/Paillier_cryptosystem#:~:text=The%20Paillier%20cryptosystem%2C%20invented%20by,believed%20to%20be%20computationally%20difficult). The sampling here does not check if this is true. There...

xiangfeimco