问题和建议

[funnywolf]

问题和建议

[FunnyWolf]

1、文件列表的下载功能有路径穿越问题
这个是我疏忽了,根本原因应该不是路径穿越问题,应该是os.path.join在处理时如果最后参数是绝对路径,会结果就是最后一个参数,也就是说不用加入../../../去穿越路径,直接使用name=/etc/passwd即可复现这个问题.
这也就是虽然我使用了filename = filename.replace("..", "")防御目录穿越,但是还是掉到了os.path.join这个坑里.
我想到的解决方式就是判断输入的路径是不是绝对路径,文件是不是存在,如果存在就返回空.
if os.path.exists(filename):
return None
应该能解决这个问题.
其实我觉得在认证情况下,viper没必要解决类似任意文件读取或者命令执行这种漏洞,因为msfconsole的存在,熟悉msf的人总能找到方法执行自定义ruby代码或者读取文件.
当然,既然提出了这个问题,就解决一下吧.
FunnyWolf/viperpython@74331ef

[FunnyWolf]

2、console是否能支持如下类似的监听方式
viper暂时不支持shell类型的session,因为几乎所有的界面功能都依赖于stdapi.

[FunnyWolf]

3、监听载荷的生成是否能支持get url下载的方式，或者生成在文件列表下方便下载
类似geturl下载方式在文件列表的一句话下载功能中有,可以将生成的payload上传到文件列表中,就点击一句话下载就会显示所有的一句话下载链接了.
至于提到的直接将生成的payload存储在文件列表中,因为当前一句话下载已经过不了任何杀软了,是否有必要呢?

[ViCrack]

@FunnyWolf
1、绝对路径的穿越问题, 既然修复了, 但修复的还不是很完美, 除了下载之外, 删除destory_msf_file, 写入或上传等操作应该也有类似的情况
而且通过判断filename路径是否存在, 来区分是否绝对路径
那么假设MSFLOOT目录下有个test文件, 根路径/下也正好有一个test文件, 这种就会冲突了
虽然可能性低...

`
MSFLOOT = "/root/xxx/"
p = os.path.join(MSFLOOT, "/../../etc/../../passwd")
print(p)
p = os.path.normpath(p)
print(p)
if p.startswith(MSFLOOT):
print("1")
else:
return None

`
或许是否可以利用normpath来判断路径前缀? 如果可行甚至连文件名中的..都不用去remove

[FunnyWolf]

@FunnyWolf
1、绝对路径的穿越问题, 既然修复了, 但修复的还不是很完美, 除了下载之外, 删除destory_msf_file, 写入或上传等操作应该也有类似的情况
而且通过判断filename路径是否存在, 来区分是否绝对路径
那么假设MSFLOOT目录下有个test文件, 根路径/下也正好有一个test文件, 这种就会冲突了
虽然可能性低...

`
MSFLOOT = "/root/xxx/"
p = os.path.join(MSFLOOT, "/../../etc/../../passwd")
print(p)
p = os.path.normpath(p)
print(p)
if p.startswith(MSFLOOT):
print("1")
else:
return None

`
或许是否可以利用normpath来判断路径前缀? 如果可行甚至连文件名中的..都不用去remove

我觉得你的方案很好,使用normpath来解决..穿越路径问题,使用os.path.join拼接之后,在使用startswith来解决os.path.join的坑.

[FunnyWolf]

FunnyWolf/viperpython@00dd5c4

[Fnzer0]

4、内网端口扫描模块，linux的权限是否有问题，我试了x86和x64的，任务是下发了，但是没结果输出，windows的权限是正常输出的

[FunnyWolf]

4、内网端口扫描模块，linux的权限是否有问题，我试了x86和x64的，任务是下发了，但是没结果输出，windows的权限是正常输出的

linux 端口扫描是有bug,现在还在修复中

[FunnyWolf]

4、内网端口扫描模块，linux的权限是否有问题，我试了x86和x64的，任务是下发了，但是没结果输出，windows的权限是正常输出的

FunnyWolf/vipermsf@c5b5f8b

[FunnyWolf]

2a133de