From: Matthieu Riou <matthieu@blockcypher•com>
To: Sharon Dvir <sharondvir@cs•huji.ac.il>
Cc: bitcoin-development@lists•sourceforge.net
Subject: Re: [Bitcoin-development] question about ProcessMessage()
Date: Thu, 17 Apr 2014 11:00:04 -0700 [thread overview]
Message-ID: <CAHUNwMoSG+C8T4_69j3-qmM_Z=WYixARYuAr1CEUv6Wxq6kajA@mail.gmail.com> (raw)
In-Reply-To: <534FA237.1010501@cs.huji.ac.il>
[-- Attachment #1: Type: text/plain, Size: 1640 bytes --]
Github has a nice feature that lets you link to a specific line of code:
https://github.com/bitcoin/bitcoin/blob/master/src/main.cpp#L3549
Is it the line you're referring to? It just flags the peer as at that point
we know we don't have more addresses to learn from it (peers send 1000
addrs). The previous
check<https://github.com/bitcoin/bitcoin/blob/master/src/main.cpp#L3497>only
applies for peers sending an addr message that would have more than
1000 addresses, which is disallowed to prevent flooding. And no number is
both greater and lesser than 1000 :)
Matthieu
On Thu, Apr 17, 2014 at 2:43 AM, Sharon Dvir <sharondvir@cs•huji.ac.il>wrote:
> Hi, in main.cpp, ProcessMessage(), the line:
> if (vAddr.size() < 1000) that appears at about line 3290
> (not sure which, as I have made changes.)
> What is the purpose of this line?
> vAddr.size() was already checked at about line 3240, and if it was >
> 1000, the function has returned.
> Is pfrom->fGetAddr allowed to be true iff vAddr.size()==1000 ?
> Could not find it in list archive.
> Thanks.
>
>
> ------------------------------------------------------------------------------
> Learn Graph Databases - Download FREE O'Reilly Book
> "Graph Databases" is the definitive new guide to graph databases and their
> applications. Written by three acclaimed leaders in the field,
> this first edition is now available. Download your free book today!
> http://p.sf.net/sfu/NeoTech
> _______________________________________________
> Bitcoin-development mailing list
> Bitcoin-development@lists•sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/bitcoin-development
>
[-- Attachment #2: Type: text/html, Size: 2423 bytes --]
prev parent reply other threads:[~2014-04-17 18:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-17 9:43 Sharon Dvir
2014-04-17 18:00 ` Matthieu Riou [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAHUNwMoSG+C8T4_69j3-qmM_Z=WYixARYuAr1CEUv6Wxq6kajA@mail.gmail.com' \
--to=matthieu@blockcypher$(echo .)com \
--cc=bitcoin-development@lists$(echo .)sourceforge.net \
--cc=sharondvir@cs$(echo .)huji.ac.il \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox