Changeset 10851 for dotorg/trunk/html/beps/bep_0008.py

Timestamp:

02/15/08 15:04:41 (2 years ago)

Author:

dave

Message:

Fix in example python that causes incorrect results when returned peer
list wraps end of tracker peer list.

Files:

• dotorg/trunk/html/beps/bep_0008.py (modified) (1 diff)

dotorg/trunk/html/beps/bep_0008.py

@@ -54 +54 @@
   response['iv'] = iv
   numwant = min(numwant, MAX_PEERS)
-  if numwant > num_peers:
+  if numwant >= num_peers:
     response['peers'] = obfuscated_tracker_peer_list
     return response
 
-  i = randint(0,num_peers)
+  i = randint(0,num_peers-numwant) 
   response['i'] = xor(i,x) 
   response['n'] = n_xor_y
-  response['peers'] = obfuscated_tracker_peer_list[i*6:(i+numwant)*6]
-  
-  if len(response['peers']) < numwant * 6:
-    r = numwant - len(response['peers']) / 6
-    response['peers'] = response['peers'] + obfuscated_tracker_peer_list[:r] 
+  # peers at end of tracker peer list have lower probability of being picked,
+  # but this requires only one copy.
+  response['peers'] = obfuscated_tracker_peer_list[i*6:(i+numwant)*6]  
   return response