我正在使用python进行一些项目。我有一个客户端和一个服务器。服务器侦听连接,一旦收到连接,它就等待来自客户端的输入。这个想法是客户端可以连接到服务器并执行系统命令,如ls和cat。这是我的服务器代码:
import sys, os, socket
host = ''
port = 50105
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
s.bind((host, port))
print("Server started on port: ", port)
s.listen(5)
print("Server listening\n")
conn, addr = s.accept()
print 'New connection from ', addr
while (1):
rc = conn.recv(5)
pipe = os.popen(rc)
rl = pipe.readlines()
file = conn.makefile('w', 0)
file.writelines(rl[:-1])
file.close()
conn.close()
这是我的客户代码:
import sys, socket
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
host = 'localhost'
port = input('Port: ')
s.connect((host, port))
cmd = raw_input('$ ')
s.send(cmd)
file = s.makefile('r', 0)
sys.stdout.writelines(file.readlines())
当我启动服务器时,我得到正确的输出,说服务器正在侦听。但是,当我与客户端连接并键入命令时,服务器退出时出现此错误:
Traceback (most recent call last):
File "server.py", line 21, in <module>
rc = conn.recv(2)
File "/usr/lib/python2.6/socket.py", line 165, in _dummy
raise error(EBADF, 'Bad file descriptor')
socket.error: [Errno 9] Bad file descriptor
在客户端,我得到了ls的输出,但服务器搞砸了。
答案 0 :(得分:6)
您的代码调用conn.close()
,然后循环回conn.recv()
,但conn
已关闭。
答案 1 :(得分:3)
如果您希望您的客户重复它正在做的事情,只需在那里添加一个循环;)
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
host = 'localhost'
port = input('Port: ')
s.connect((host, port))
while True:
cmd = raw_input('$ ')
s.send(cmd)
file = s.makefile('r', 0)
sys.stdout.writelines(file.readlines())
应该更接近你想要的。
其他评论:
s.listen(1)
此语句可能应该移到while循环之外。您只需拨打listen
一次。
pipe = os.popen(rc)
os.popen已被弃用,请改用subprocess module。
file = s.makefile('r', 0)
您正在打开文件,但您永远不会关闭该文件。您应该在file.close()
来电后添加sys.stdout.writelines()
。
编辑:回答以下评论;由于长度和格式在这里完成
就目前而言,您只需从插座读取一次,然后立即关闭它。因此,当客户端发送下一个命令时,它会看到服务器关闭套接字并指示错误。
解决方案是更改您的服务器代码,以便它可以处理接收多个命令。请注意,这是通过引入另一个循环来解决的。
你需要换行
rc = conn.recv(2)
pipe = os.popen(rc)
rl = pipe.readlines()
fl = conn.makefile('w', 0)
fl.writelines(rl[:-1])
在另一个while True:
循环中,以便它重复,直到客户端断开连接,然后将其包装在try-except块中,该块捕获客户端断开连接时conn.recv()抛出的IOError。
try-except块应该看起来像
try:
# the described above loop goes here
except IOError:
conn.close()
# execution continues on...