2017-03-06 17 views
0

私はまだ学生ですので、本当に明白な何かを見逃しているかもしれません。私の些細なtftpクライアントの背後にあるロジックに欠陥がありますか?

だから私はこのことについて非常に強調しています。

しかし、とにかくここに私のTFTPのpythonコードがあります、それは私たちの講師のサーバーからテキストファイルをダウンロードしています。

それによって生成されたファイルは次のようになります。 http://pastebin.com/TP8hngxM

そして、このような元のファイル: http://pastebin.com/xDMjkABp

そして、あなたは違いチェッカーを通してそれを実行した場合、差は非常に小さいとのみ1でありますこれがなぜ起こっているのかを理解することは本当に難しいです。

ダウンロードしたファイルには、いくつかの単語が追加されています。

余分な5分があれば、間違っているかどうかを確認するために、入れ子にされたwhileループをチェックしてください(他のものは講師によって提供され、変更できません)。

最悪の事は私がすでにそれを働かせていたことですが、メモリスティックがなくなり、100%正常に動作していたプログラムの最新バージョンが失われました。

私が言ったように、それはネストされたwhileループに関するだけです。私はそれ以上のものを変更することはできません。

#!/usr/bin/python 

import struct 
import sys 
import os 
import select 
import glamnetsim 
from socket import * 

serverHost = 'mcgreg.comp.glam.ac.uk' 
serverPort = 69 
timeoutSecs = 5 
debugging = False 
RRQ, WRQ, DATA, ACK, ERROR = range(1, 6) 
codeDescriptions = {RRQ:"RRQ", WRQ:"WRQ", DATA:"DATA", ACK:"ACK", ERROR:"ERROR"} 

def printf (format, *args): 
    print str(format) % args, 

def finish(): 

    printf("you should see\n1e951df315d433aa4df2065a1ad31311\n") 
    os.system("md5sum newfile.txt") 
    sys.exit(0) 

def sendPacket (packet, port): 
    global sock, debugging 
    global serverIp 
    if debugging: 
     for i in packet: 
      print ('%02x' % ord(i)), 
     print '' 
    sock.sendto(packet, (serverIp, port)) 

def sendReadRequest (filename, mode): 
    global serverPort 
    format = "!H%ds" % (len(filename)+1) 
    format += "%ds" % (len(mode)+1) 
    s = struct.pack(format, 1, filename, mode) 
    sendPacket(s, serverPort) 

def sendRealAck(blockno, port): 
    s = struct.pack("!H", 4) 
    s += struct.pack("!H", blockno) 
    sendPacket(s, port) 

def sendACK (blockno, port): 
    print " -> ACK:%d\n" % blockno 
    if blockno == 0: 
     sendReadRequest("the_machine_stops.txt", "octet") 
    else: 
     sendRealAck(blockno, port) 

def stripPacket (s): 
    if len(s)>3: 
     code = struct.unpack("!H", s[:2])[0] 
     blockno = struct.unpack("!H", s[2:4])[0] 
     data = s[4:] 
     code, data = glamnetsim.simulatePossibleError (code, data) 
     return code,blockno,data 
    else: 
     debugPrint("corrupt packet") 
     return -1,-1,"" 

     def debugPrint (s): 
    global debugging 
    if debugging: 
     print s 

def getDesc (c): 
    global codeDescriptions 
    return codeDescriptions[c] 

sock = socket(AF_INET, SOCK_DGRAM) 
serverIp = gethostbyname(serverHost) 
sock.setblocking(1) 
sendReadRequest("the_machine_stops.txt", "netascii") 
lastblock = 0 
blockno = 0 
port = serverPort 
f = open("newfile.txt", "w") 

while True: 
    while True: 

     if blockno == lastblock+1: 
       break 

     r, w, x = select.select([sock], [], [], 5.0) 

     if r == []: 
      sendACK(lastblock, port) 

     else: 

      (packet, (address, port)) = sock.recvfrom(512+4) 
      code, newblock, text = stripPacket(packet) 
      print code, blockno   

      if code is 3: 
       blockno = newblock 
       sendACK(blockno, port) 

      if code is 5: 
       sendACK(lastblock, port) 

      print "Bn: " + str(blockno) + " Lb: " + str(lastblock) 
    lastblock = blockno 
    f.write(text) 
    print "OK" 
    if len(text) < 512: 
     break 
f.close() 
finish() 
+0

私はこれを理解しようとしています。「if code is 5:」が発生したときに中断しなければならないと感じています。あなたはどう思う? –

+0

私はこれを再現できないと思います。 'glamnetsim'とは何ですか? – nickie

+0

これは大学の内部ネットワークをシミュレートする関数です。ファイアウォールの背後にあるので、私のコードを実行することはできません。接続するにはキャンパスにいなければなりません。私は、プログラム自体のロジックに何か問題があるかどうかを判断しようとしています。 –

答えて

0

blockno == lastblock+1場合のラインが古い値で実行されていることをf.write(テキスト)

。おそらく内側のループ内にあるはずです。

関連する問題