)]}'
{
  "commit": "d5e4586546974179feca305a94e07fac3e9727fe",
  "tree": "7cdd8fff16e52e08cf171f375a97ceadd7419297",
  "parents": [
    "d0e69d9afa59b93c30294eba89b1f15f69e91105"
  ],
  "author": {
    "name": "Zhao Li",
    "email": "enderaoelyther@gmail.com",
    "time": "Tue Jul 07 10:53:34 2026 +0800"
  },
  "committer": {
    "name": "Johannes Berg",
    "email": "johannes.berg@intel.com",
    "time": "Tue Jul 07 09:29:26 2026 +0200"
  },
  "message": "wifi: cfg80211: validate rx/tx MLME callback frame lengths before access\n\ncfg80211_rx_mlme_mgmt() and cfg80211_tx_mlme_mgmt() call tracepoints\nbefore rejecting frames shorter than the frame-control field. After\nthat, they only require len \u003e\u003d 2 before dispatching into subtype\nhandlers that assume their fixed fields are present.\n\nThe frames that trip this are not shorter than 2 bytes; they are short\nrelative to their subtype. mwifiex is a concrete in-tree example on the\nlength side: mwifiex_process_mgmt_packet() only requires a 4-address\nieee80211_hdr plus the 2-byte firmware length prefix before handing the\nframe to cfg80211_rx_mlme_mgmt(). After stripping the length prefix and\nremoving addr4, pkt_len can be exactly 24: a bare 3-address management\nheader with no reason-code body. The existing WARN_ON(len \u003c 2) does not\nfire on such a frame, and cfg80211_process_deauth() then reads\nu.deauth.reason_code as a two-byte access starting at offset 24,\nimmediately past the 24-byte buffer.\n\nAdd a frame-control length gate, then validate each subtype\u0027s minimum\nframe size in an if/else-if chain that mirrors the dispatch logic. Trace\nonly after the frame is known to be well-formed.\n\nSide effects of this change:\n - The WARN_ON(len \u003c 2) is dropped. It only guarded the frame_control\n   read, never the subtype fixed fields, and it does not fire on the\n   frames that actually trigger the out-of-bounds read (which are \u003e\u003d 2).\n   The len \u003e\u003d 2 check is kept as the guard before dereferencing\n   frame_control, but without the warning: these are exported callbacks\n   and a malformed frame from a driver should be dropped silently rather\n   than backtraced.\n - cfg80211_tx_mlme_mgmt() previously routed every non-deauth subtype\n   through disassociation handling; it now silently ignores unrecognised\n   subtypes.\n\nAssisted-by: Codex:gpt-5.5\nAssisted-by: Claude:claude-opus-4.8\nSigned-off-by: Zhao Li \u003cenderaoelyther@gmail.com\u003e\nLink: https://patch.msgid.link/20260707025336.22557-1-enderaoelyther@gmail.com\nSigned-off-by: Johannes Berg \u003cjohannes.berg@intel.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "2a2c173058bacada2f36a05f88742b487e8f60c8",
      "old_mode": 33188,
      "old_path": "net/wireless/mlme.c",
      "new_id": "acc749eee2fea8579fef853003311898b809575f",
      "new_mode": 33188,
      "new_path": "net/wireless/mlme.c"
    }
  ]
}