smooth
Legendary
Offline
Activity: 2968
Merit: 1198
|
|
May 01, 2014, 12:38:04 AM |
|
oaes_key_gen() does not seem to be used anywhere, and the function is not documented, but why it uses rand() to generate the key? Dumb, but there is good code to generate keys using an entropy source somewhere else. (I forgot where but I did review it.) Someone should comment out oaes_key_gen and recompile to make sure it really isn't being used.
|
|
|
|
otila
|
|
May 01, 2014, 07:30:03 AM |
|
oaes_key_gen() does not seem to be used anywhere, and the function is not documented, but why it uses rand() to generate the key? Dumb, but there is good code to generate keys using an entropy source somewhere else. (I forgot where but I did review it.) Someone should comment out oaes_key_gen and recompile to make sure it really isn't being used. Did that, programs linked OK.
|
|
|
|
digicoin
Legendary
Offline
Activity: 1106
Merit: 1000
|
|
May 01, 2014, 07:20:08 PM |
|
The public key in CryptoNote is way too long. Should we make it shorter or it is on purpose?
It reminds me of Bitcoin address. It is a hashed version of the public key. The public key is 256 bits long, and the final hash (ie. address) is 160 bits long.
I am little bit curious about CryptoNote's technical background of an address
PS: Sorry, I haven't read the code yet
|
|
|
|
tacotime
Legendary
Offline
Activity: 1484
Merit: 1005
|
|
May 01, 2014, 07:26:11 PM |
|
The public key in CryptoNote is way too long. Should we make it shorter or it is on purpose?
It reminds me of Bitcoin address. It is a hashed version of the public key. The public key is 256 bits long, and the final hash (ie. address) is 160 bits long.
I am little bit curious about CryptoNote's technical background of an address
PS: Sorry, I haven't read the code yet
The address are similar in size to those of Bitcoin stealth addresses. There is no conceivable way to shorten them that would lot lead to attacks through ECDH. I would suggest you read up on stealth addressing.
|
XMR: 44GBHzv6ZyQdJkjqZje6KLZ3xSyN1hBSFAnLP6EAqJtCRVzMzZmeXTC2AHKDS9aEDTRKmo6a6o9r9j86pYfhCWDkKjbtcns
|
|
|
|
digicoin
Legendary
Offline
Activity: 1106
Merit: 1000
|
|
May 02, 2014, 09:45:29 AM |
|
I think that we need to mailboxd to read and store such alert history. Is such alerts stored in blockchain?
|
|
|
|
crypto_zoidberg (OP)
|
|
May 02, 2014, 07:28:01 PM Last edit: May 02, 2014, 09:13:00 PM by crypto_zoidberg |
|
I think that we need to mailboxd to read and store such alert history. Is such alerts stored in blockchain?
No. There are few abstraction layers in CryptoNote code(as described on our page http://honeypenny.org/design.html), Blockchain lies on upper level. We've done this feature on p2p level in two reasons: 1. P2p code could be taken for another project based on p2p technology, so this feature would be useful. 2. Having alerts based on blockchain makes more complicated to push alert devs have to look for some of miners and ask them to do it. Anyway, collecting this alerts archive may be done with local daemon storage.
|
|
|
|
digicoin
Legendary
Offline
Activity: 1106
Merit: 1000
|
|
May 02, 2014, 09:22:49 PM |
|
Is it possible to use this feature to send a p2p message from user A to user B where the message is encrypted, unfakable?
|
|
|
|
crypto_zoidberg (OP)
|
|
May 03, 2014, 09:15:23 AM |
|
Is it possible to use this feature to send a p2p message from user A to user B where the message is encrypted, unfakable?
Unfortunately no.
|
|
|
|
crypto_zoidberg (OP)
|
|
May 03, 2014, 11:12:41 PM |
|
Good news! We've added UPnP support with reannouncing every 20 minutes (like bitcoin do). Just merged it in master branch. Anyone who have UPnP router - please check this - or by running daemon, or just run functional_tests --test_miniupnp. We've checked it on two devices - one was correctly configured, other failed.
|
|
|
|
emontmon
Member
Offline
Activity: 196
Merit: 10
|
|
May 05, 2014, 03:12:29 PM Last edit: May 05, 2014, 06:39:57 PM by emontmon |
|
can you change the op regarding network launch
1. will be launched with prior notice of two days. 2. will launch network with 2 days prior notice. estimated date
I keep thinking I missed it as it says this week as of last week. So i guess I am not sure that you launched already
|
|
|
|
otila
|
|
May 05, 2014, 10:17:39 PM |
|
What's the purpose of mul_f::keccakf_m function, keccak modified to use multiplying in bc[5] setup?
|
|
|
|
superresistant
Legendary
Offline
Activity: 2156
Merit: 1131
|
|
May 06, 2014, 08:42:01 AM |
|
The name is quite ridiculous IMO.
It's funny though.
|
|
|
|
otila
|
|
May 06, 2014, 01:41:10 PM |
|
[ 42%] Building CXX object src/CMakeFiles/crypto.dir/crypto/keccak_modified.cpp.o In file included from /c/honeypenny/src/crypto/keccak_modified.cpp:6:0: /c/honeypenny/src/crypto/keccak_modified.h: In function ‘pod_operand_a crypto::xor_pod(const pod_operand_a&, const pod_operand_b&)’: /c/honeypenny/src/crypto/keccak_modified.h:30:5: error: ‘hash’ is not a member of ‘crypto’ crypto::hash r; ^
|
|
|
|
|
otila
|
|
May 07, 2014, 04:18:36 PM |
|
tried with empty ~/.HoneyPenny 2014-May-07 19:11:12.235978 [P2P7]ERROR /c/honeypenny/src/currency_core/blockchain_storage.cpp:1139 Client sent wrong NOTIFY_REQUEST_CHAIN: genesis block missmatch: id: <830ec1d9917286cd95ac0128022900dc452c33d9d29974ae304bff430fc10353>, expected: <6d0a2fd27d7ba5867d966528d7ac0334df54ef38b1826eab9e651431e7d7a847>, dropping connection 2014-May-07 19:11:12.236062 [P2P7]ERROR /c/honeypenny/src/currency_protocol/currency_protocol_handler.inl:365 [107.170.228.11:10101 OUT]Failed to handle NOTIFY_REQUEST_CHAIN.
|
|
|
|
TheMightyX
Sr. Member
Offline
Activity: 350
Merit: 250
Vires in Numeris
|
|
May 07, 2014, 04:32:30 PM |
|
Can we get an update on the announcement of launch?
The OP says April 27 and launch will be "this week", which it obviously wasn't.
Please be more clear. Thanks
|
|
|
|
crypto_zoidberg (OP)
|
|
May 07, 2014, 07:45:43 PM |
|
What's the purpose of mul_f::keccakf_m function, keccak modified to use multiplying in bc[5] setup?
Little surprise for ASIC engineers, considered that effective 64-bit multiplication is pretty expensive to have on each core. (unlike normal CPU) Just to reduce the enthusiasm.
|
|
|
|
crypto_zoidberg (OP)
|
|
May 07, 2014, 07:48:16 PM |
|
When its launching? Will you chose the new name right away?
11-12 May
|
|
|
|
emontmon
Member
Offline
Activity: 196
Merit: 10
|
|
May 07, 2014, 07:50:05 PM |
|
Can this be mined with the galaxy s5
|
|
|
|
|