-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Idea: correction words for truncated weights #78
Comments
How about:
|
I think that works. I believe we'll need to invoke It's sad that this means I wonder if we could do something like this: include shares of |
Just to rephrase, the VIDPF Theoretically, this could work and keep VIDPF as a black box but we need to be sure we cannot find |
Excellent point. I think this property ought to hold for all weight types we're considering so far. |
Almost done but I ran into an issue. In |
Hmmmm yeah you're right. I don't think that quite works. |
I'm not sure we should closet his -- It seems like we can't make this black box, but I think it might still worth doing. |
(I'd like to keep this open until I've had time to think it through.) |
The idea of #109 is to apply VIDPF to the truncated weight and FLP to the untruncated weight. We would "link" them by incorporating the untruncated weight into the payload check. This would allow for the following "attack": a client can modify a report by "resharing" the FLP weight shares, and the aggregators would accept the modified report. This isn't an attack against our current definition of robustness (ia.cr/2023/130), as the aggregators would still compute the same output shares. However, we're worried this could constitute an attack in the higher level protocol. Our current approach isn't vulnerable to this attack because the sharing of the weight is determined by the VIDPF. |
The FLP is run on the encoded measurement, which in general is longer than the truncated measurement. In some cases it might be much longer, e.g., for the
Sum
circuit, the encoded measurement is several field elements, while the truncated measurement is just one.The encoded measurement is only needed when we verify the FLP. Suppose we modified Mastic as follows:
flp.encode(weight)
, in the input shares.beta
to be the encoded measurement share, setbeta
to be the truncated measurement, i.e.,beta = flp.truncate(flp.encode(weight))
.The text was updated successfully, but these errors were encountered: