osu-crypto / libPSI

A repository for private set intersection.
Other
168 stars 47 forks source link

Conversion Error #51

Closed h4ck4t closed 1 year ago

h4ck4t commented 1 year ago

System: Windows10 IDE: Visual Studio 2022 Problem: When I try to compile the project, following problems show up, and I'm confused that mOtSender->recvCorrection method takes a Channel as input, while its definition is Socket.

Severity Code Description Project File Line Suppression State Error C2664 'macoro::task<void,true> osuCrypto::NcoOtExtSender::recvCorrection(osuCrypto::Socket &,osuCrypto::u64)': cannot convert argument 1 from 'T' to 'osuCrypto::Socket &' libPSI C:\Users\ceezous\Desktop\libPSI-private\libPSI\PSI\Kkrt\KkrtPsiSender.cpp 374

Severity Code Description Project File Line Suppression State Error C2664 'macoro::task<void,true> osuCrypto::NcoOtExtSender::init(osuCrypto::u64,osuCrypto::PRNG &,osuCrypto::Socket &)': cannot convert argument 3 from 'T' to 'osuCrypto::Socket &' libPSI C:\Users\ceezous\Desktop\libPSI-private\libPSI\PSI\Kkrt\KkrtPsiSender.cpp 86

Severity Code Description Project File Line Suppression State Error C2664 'macoro::task<void,true> osuCrypto::NcoOtExtReceiver::sendCorrection(osuCrypto::Socket &,osuCrypto::u64)': cannot convert argument 1 from 'T' to 'osuCrypto::Socket &' libPSI C:\Users\ceezous\Desktop\libPSI-private\libPSI\PSI\Kkrt\KkrtPsiReceiver.cpp 189

Severity Code Description Project File Line Suppression State Error C2664 'macoro::task<void,true> osuCrypto::NcoOtExtReceiver::init(osuCrypto::u64,osuCrypto::PRNG &,osuCrypto::Socket &)': cannot convert argument 3 from 'T' to 'osuCrypto::Socket &' libPSI C:\Users\ceezous\Desktop\libPSI-private\libPSI\PSI\Kkrt\KkrtPsiReceiver.cpp 105

ladnir commented 1 year ago

You have the wrong version of libOTe. Did you install libOTe on your own?

h4ck4t commented 1 year ago

Yeah. I tried python.py,but seems having difficulty cloning libote in my area. So I compiled manually.

Is there any version indication?

Get Outlook for iOShttps://aka.ms/o0ukef


From: Peter Rindal @.> Sent: Wednesday, November 30, 2022 8:00:10 AM To: osu-crypto/libPSI @.> Cc: maplepicker @.>; Author @.> Subject: Re: [osu-crypto/libPSI] Conversion Error (Issue #51)

You have the wrong version of libOTe. Did you install libOTe on your own?

— Reply to this email directly, view it on GitHubhttps://github.com/osu-crypto/libPSI/issues/51#issuecomment-1331472936, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A32TYW7PK3ANC65JHX6WNPLWK2KIVANCNFSM6AAAAAASOUKL2U. You are receiving this because you authored the thread.Message ID: @.***>

ladnir commented 1 year ago

https://github.com/osu-crypto/libPSI/blob/2eb0514f66a0a0d8ce0ffcead73485886f21306f/thirdparty/getLibOTe.cmake#L4

LibOTe v1.6 should also work but the link above has the currently tested git commit.

h4ck4t commented 1 year ago

https://github.com/osu-crypto/libPSI/blob/2eb0514f66a0a0d8ce0ffcead73485886f21306f/thirdparty/getLibOTe.cmake#L4

LibOTe v1.6 should also work but the link above has the currently tested git commit.

Thanks for suggestion. Now I found that libOTe being configured properly, but it's relic that makes things complex. The information is: Build

'C:/Program Files/CMake/bin/cmake.exe' '--build' 'C:/Users/ceezous/Desktop/libPSI-private/thirdparty/libOTe/cryptoTools/thirdparty/relic/build/x64-Release' '--config' 'Release'

CMake Error at cryptoTools/thirdparty/fetch.cmake:35 (message):

Build failed (1).
ladnir commented 1 year ago

So libOTe is also going to pull in a bunch of things. Relic etc...

You will need to clone these manually as well. Or build the whole thing with internet and then move it off internet...