04:19:12 thanks wowmario 04:19:28 *wowario 12:28:22 why are key images part of the tx prefix hash? Given that signatures offer the key image anyway 12:28:53 is it just for convenience in the code 12:29:01 simplicity 13:32:38 If I wanted to take a random hash (anything hash possible in the 32 byte space) and turn it into a valid EC point, which function should be used? 13:33:06 is 13:33:06 ge_fromfe_frombytes_vartime 13:33:07 sufficient? 13:34:41 or will that only work for valid hashes of *allowed *curve points 13:35:12 hashToPoint IIRC. 13:35:46 Basically hashes, multiplies by G and by 8. 13:36:52 You may want to add some "domain salt" to the data you hash. 13:41:16 when you say it hashes, do you mean it hashes and converts to scalar modulo l 13:41:25 then does * G * 8 13:41:45 the function reads: 13:41:46 void hash_to_point(const crypto::hash &h, crypto::ec_point &res) { 13:41:46   ge_p2 point; 13:41:47   ge_fromfe_frombytes_vartime(&point, reinterpret_cast(&h)); 13:41:47   ge_tobytes(crypto::operator &(res), &point); 13:41:48 } 13:46:05 Looks like I remembered wrong, hashToPoint does not exist... 13:47:48 doesn't the one above do the trick? 13:49:08 Ah, it got removed. There's hash_to_p3 now. 13:49:28 And it looks like it's doing what you say. Plus the *8. 13:53:06 am I right in thinking the p2 is the compressed point and p3 has the explicit sign 13:53:14 thanks btw will take a look 14:01:46 Would you mind explaining what you meant by domain salt? 14:01:50 in this context anyway 14:02:43 Some arbitrary data concatenated to your input, such that the same input will not yield hte same point for two different uses. Might or might not be useful in your case. 14:05:37 thanks but doesn't affect me 15:02:01 peach34: key images are in the prefix explicitly, as opposed to implicitly in the round harsh 15:02:05 hash* 15:04:00 it’s probably more of a best practice than absolutely necessary 15:05:42 yes that's what I meant 21:24:28 .merges 21:24:28 Merge queue empty 21:25:13 .merge+ 7308 7309 21:25:14 Added 21:26:03 7309 needs an approval, 7308 (master equivalent) is approved by vtnerd