Tuesday, 2014-09-16

*** tpb has joined #pettycoin00:00
*** rusty has joined #pettycoin00:59
rustyarhuaco: found that bug, thanks.02:26
rustyarhuaco: we have a new one though... somehow the same tx appeared in blocks 94 and 95.  Restarting the client complains, but petty1 is convinced it's OK.02:26
arhuacorusty: I see... Well, one bug less... I think I caused it because I stop pettycoin with Ctrl+C.02:30
arhuacorusty: I mean issues/33.02:32
rustyarhuaco: no, 33 is caused by my dumb typo.02:32
rustyarhuaco: so now I've fixed that, but am opening two more bugs :)02:34
githubby[pettycoin] rustyrussell pushed 3 new commits to master: http://git.io/RElUhA02:34
githubbypettycoin/master f99b1ab Rusty Russell: chain: fix check_chains() (Fixes #33)...02:34
githubbypettycoin/master 727ec75 Rusty Russell: block_info: remove unused prevs field....02:34
githubbypettycoin/master 6851d62 Rusty Russell: recv_block: ask for block contents on reinject....02:34
arhuacorusty: He he.02:34
rustyarhuaco: but at least it syncs really fast now, before it complains and disconnects :)02:35
arhuacorusty: block_shard.c:81: check_block_shard: Assertion `!shard->proof' failed02:36
arhuacorusty: Shall I sync all the blocks again?02:37
rustyarhuaco: oh, cool!  I haven't seen that.  I'll grab your block file and try to reproduce....02:37
arhuacorusty: OK.02:37
arhuacorusy: I switched to master, rebuilt, and ran it...02:37
rustyarhuaco: BTW, looks like you did a doublespend :)02:40
arhuacorusy: I have talent for this :-P02:41
rustyarhuaco: it's great!  Other nodes rejected it, but of course petty1 should never have allowed it.02:41
rustybug #34 has all the gory details.02:42
rustyarhuaco: if I restart pettycoin it will detect the doublespend and start mining before it, but I want to figure out how it happened.02:42
githubby[pettycoin] rustyrussell pushed 1 new commit to master: http://git.io/2etedw02:51
githubbypettycoin/master f95349d Rusty Russell: check_tx: return correct input error....02:51
rustyarhuaco: OK, with that commit, nodes correctly ignore blocks 95 onwards (which has the doublespend).02:51
rustyarhuaco: of course, petty1 doesn't believe the complaints and sends an error packet :)02:52
*** arhuaco has quit IRC02:53
githubby[pettycoin] rustyrussell pushed 1 new commit to master: http://git.io/lIhVDQ03:03
githubbypettycoin/master ffedf26 Rusty Russell: check_block: free up proofs when we add txhash....03:03
*** ChanServ sets mode: +o rusty03:03
*** rusty changes topic to " Test network v-alpha02 (or pref newer) || http://pettycoin.org/ || Logs @ http://logs.timvideos.us/%23pettycoin/ || Gateway status: http://128.199.137.156/"03:04
*** rusty sets mode: -o rusty03:04
*** arhuaco has joined #pettycoin03:07
rustyarhuaco: your bug is fixed... now to the ds :)03:19
arhuacorusty: Good.03:19
rustyarhuaco: I think I know what it is, but I'm writing a new testcase, because it's kind of an important bug :)04:11
arhuacorusty: Wow. I remember I did two transfers, each of them with more than half the available amount. Is this something similar?04:13
rustyarhuaco: looks like you tried to respend the same transfer.04:14
rustyarhuaco: the network should have ignored the second one, but because of a bug, it didn't.04:14
arhuacorusty: I see. At some point I think I saw a warning saying that I had already spent a transaction. Maybe I only got the warning but the limit was not enforced?04:15
rustyarhuaco: well, somehow petty1 accepted the second one.  It's a good test, and nice to know the rest of the network rejected the lobkc.04:16
githubby[pettycoin] rustyrussell pushed 2 new commits to master: http://git.io/SL7acQ05:26
githubbypettycoin/master bdca104 Rusty Russell: recv_block.h: remove seek_detached_blocks() prototype....05:26
githubbypettycoin/master 8fe6c66 Rusty Russell: recv_block: don't segv on generation or json inject....05:26
githubby[pettycoin] rustyrussell pushed 3 new commits to master: http://git.io/1TXCHw07:13
githubbypettycoin/master 0cb301a Rusty Russell: reward: fix reward modulus calculation....07:13
githubbypettycoin/master 96732e7 Rusty Russell: pettycoin-generate: seperate out make_shard_pkt function for easier testing....07:13
githubbypettycoin/master db9bfa2 Rusty Russell: inputhash: fix use-after-free which allowed doublespend (Fixes #34)...07:13
*** arhuaco has quit IRC07:37
githubby[pettycoin] rustyrussell pushed 1 new commit to master: https://github.com/rustyrussell/pettycoin/commit/2b547bae2af24b70fc0c353749b4ffbf6d04a53a07:41
githubbypettycoin/master 2b547ba Rusty Russell: complain.c: fix complaint on incomplete blocks....07:41
*** rusty has quit IRC07:49
*** arhuaco has joined #pettycoin07:50
*** rusty has joined #pettycoin09:28
*** rusty has quit IRC12:42
*** rusty has joined #pettycoin14:11
*** rusty has quit IRC15:56

Generated by irclog2html.py 2.12.1 by Marius Gedminas - find it at mg.pov.lt!