如何突破JFinal黑名单机制实现任意文件上传

readNextPart() //轮询处理从上传文件的时候前端传来的参数

跟进代码的时候发现，系统中处理上传文件的代码是如下形式：

UploadFile file = getFile("file", "test");

看了一下lib，发现这个函数是JFinal框架处理上传文件的函数，那是不是代表这种利用方式是在JFinal框架中是具有通用性的

所以直接在本地搭了一个JFinal框架去看一下上传文件的代码。

首先去看一下黑名单处理函数

Upload/MultipartRequest.class  100-107

private boolean isSafeFile(UploadFile uploadFile) {

    String fileName = uploadFile.getFileName().trim().toLowerCase();

    if (!fileName.endsWith(".jsp") && !fileName.endsWith(".jspx")) {

        return true;

    } else {

        uploadFile.getFile().delete();

        return false;

    }

}

可以发现在处理后缀为jsp和jspx类型的文件的时候，会做一步删除操作“uploadFile.getFile().delete(); “

看到这一步的时候我就基本心如明镜了，在这个地方做了删除操作是不是就代表了，代码在处理上传文件的时候先是什么都不管的，上传完成后再对有问题的文件做删除。

那我们先去跟一下上传文件的代码

Servlet/MultipartRequest.class   84-108
while((part = parser.readNextPart()) != null) {

    String name = part.getName();

    if (name == null) {

        throw new IOException("Malformed input: parameter name missing (known Opera 7 bug)");

    }



    String fileName;

    if (part.isParam()) {

        ParamPart paramPart = (ParamPart)part;

        fileName = paramPart.getStringValue();

        existingValues = (Vector)this.parameters.get(name);

        if (existingValues == null) {

            existingValues = new Vector();

            this.parameters.put(name, existingValues);

        }



        existingValues.addElement(fileName);

    } else if (part.isFile()) {

        FilePart filePart = (FilePart)part;

        fileName = filePart.getFileName();

        if (fileName != null) {

            filePart.setRenamePolicy(policy);

            filePart.writeTo(dir);

            this.files.put(name, new UploadedFile(dir.toString(), filePart.getFileName(), fileName, filePart.getContentType()));

        }

可以看到代码会去轮询上传时候传的每个参数，一旦是文件，就会直接上传到给定的dir。

现在我们已经明确了，无论什么类型的文件都会上传，但是在抵达黑名单函数后就会对jsp和jspx文件进行删除

所以我们接下来要思考的问题就是：

如何让代码执行到上传文件后，而又不去执行黑名单处理代码

总结刚刚我们在测试阶段的分析，比较良好的做法就是想办法让程序执行到黑名单函数之前报错，这样代码就会停止执行到黑名单函数。

那我们去看一下代码，

Upload/MultipartRequest.class  76-87

this.multipartRequest = new com.oreilly.servlet.MultipartRequest(request, uploadPath, maxPostSize, encoding, fileRenamePolicy);

Enumeration files = this.multipartRequest.getFileNames();



while(files.hasMoreElements()) {

    String name = (String)files.nextElement();

    String filesystemName = this.multipartRequest.getFilesystemName(name);

    if (filesystemName != null) {

        String originalFileName = this.multipartRequest.getOriginalFileName(name);

        String contentType = this.multipartRequest.getContentType(name);

        UploadFile uploadFile = new UploadFile(name, uploadPath, filesystemName, originalFileName, contentType);

        if (this.isSafeFile(uploadFile)) {

            this.uploadFiles.add(uploadFile);

        }

    }

}

从上传文件

this.multipartRequest = new com.oreilly.servlet.MultipartRequest(request, uploadPath, maxPostSize, encoding, fileRenamePolicy);

到删除黑名单文件

this.isSafeFile(uploadFile)

这段代码之间我们还是有很多操作可以去做的。

我这边的思路就是，首先保证上传文件的参数正常，再在文件参数后去给一个该接口不存在的参数

于是在轮询上传参数的时候

第一个参数是文件，直接会被上传到服务器

而去轮询第二个参数的时候，系统就会“报参数不存在“的异常，从而停止程序的执行，绕过黑名单机制。

看到这些代码也可以明白，第一版的时候LuoKe同学绕过黑名单成功的原因是因为提交的第二个参数缺少分割boundary导致的报错，思路同样可行。

下图给出poc：

看一下后台抛的异常：