Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: github.com/usmannasir/cyberpanel
Language: Python
File: databases/views.py
Function: restartMySQL

Call path: restartMySQL (databases/views.py) → mysqlUtilities.restartMySQL (plogical/mysqlUtilities.py) → ACLManager.loadedACL (plogical/acl.py) → ACLManager.loadErrorJson (plogical/acl.py)

### Primary Function

```python
def restartMySQL(request):
    try:
        userID = request.session['userID']
        finalData = mysqlUtilities.restartMySQL()
        currentACL = ACLManager.loadedACL(userID)

        if currentACL['admin'] == 1:
            pass
        else:
            return ACLManager.loadErrorJson('FilemanagerAdmin', 0)

        data = {}

        data['status'] = finalData[0]
        data['error_message'] = finalData[1]
        json_data = json.dumps(data)
        return HttpResponse(json_data)

    except KeyError:
        return redirect(loadLoginPage)
```

### Cross-File Context

[mysqlUtilities — module — databases/views.py:16]
from plogical.mysqlUtilities import mysqlUtilities

[ACLManager — module — databases/views.py:13]
from plogical.acl import ACLManager

[ACLManager.loadedACL — helper — plogical/acl.py]
ACLManager.loadedACL(userID)

[ACLManager.loadErrorJson — helper — plogical/acl.py]
ACLManager.loadErrorJson('FilemanagerAdmin', 0)

Control flow: The request enters restartMySQL, extracts userID from session, calls mysqlUtilities.restartMySQL() immediately without authorization check, then checks if user is admin and returns error if not. The fix reorders to check admin status first before calling restartMySQL.

Data flow: userID flows from request.session to ACLManager.loadedACL. currentACL dict determines authorization. The vulnerable flow has restartMySQL() execute regardless of admin status.

Verdict: Vulnerable
Vulnerable region:
[databases/views.py]
```python
def restartMySQL(request):
    try:
        userID = request.session['userID']
        finalData = mysqlUtilities.restartMySQL()
        currentACL = ACLManager.loadedACL(userID)

        if currentACL['admin'] == 1:
            pass
        else:
            return ACLManager.loadErrorJson('FilemanagerAdmin', 0)

        data = {}

        data['status'] = finalData[0]
        data['error_message'] = finalData[1]
        json_data = json.dumps(data)
```
CWE: CWE-862
Severity: MEDIUM

Root cause: In databases/views.py restartMySQL function, the mysqlUtilities.restartMySQL() call is placed before the ACL admin verification check. Any authenticated user can invoke this endpoint and restart MySQL, even if they lack admin privileges.

Exploitability: An authenticated user with a valid session can call the /databases/restartMySQL endpoint and trigger MySQL restart without admin privileges.

Attack preconditions:
- Attacker must have an active user session in CyberPanel
- Attacker must be able to send HTTP requests to the vulnerable endpoint

Likely impact: An authenticated user with a valid session can call the /databases/restartMySQL endpoint and trigger MySQL restart without admin privileges.

An analysis of the provided code reveals a significant authorization bypass vulnerability due to the incorrect ordering of operations.

### Vulnerability Analysis

In the `restartMySQL` function, the application intends to restrict the ability to restart the MySQL service to administrators only. However, the authorization check is performed **after** the sensitive action has already been executed.

Observe the order of execution in the code:
```python
def restartMySQL(request):
    try:
        userID = request.session['userID']
        # 1. The sensitive action is executed first.
        finalData = mysqlUtilities.restartMySQL()

        # 2. The authorization check is performed afterwards.
        currentACL = ACLManager.loadedACL(userID)

        if currentACL['admin'] == 1:
            pass
        else:
            # 3. An error is returned, but the action has already occurred.
            return ACLManager.loadErrorJson('FilemanagerAdmin', 0)
```

Because `mysqlUtilities.restartMySQL()` is called before `currentACL['admin'] == 1` is evaluated, the MySQL service will be restarted regardless of the user's privilege level. The subsequent `if/else` block only controls whether the user sees a success message or an error message, but it fails to prevent the action itself.

### Root Cause
The root cause is an Incorrect Behavior Order (CWE-696) leading to Improper Authorization (CWE-285). The system relies on a post-execution authorization check, which is entirely ineffective at stopping unauthorized actions.

### Attack Preconditions
The attacker only needs to be authenticated (i.e., have a valid `userID` in their session), which implies any low-privileged user account can trigger this vulnerability.

### Likely Impact
Any authenticated user can restart the MySQL database service on the server at will. Repeatedly exploiting this could lead to a Denial of Service (DoS) condition for any applications or services relying on the database, by constantly taking the database offline.

### Verdict
The code is **Vulnerable**. The sensitive database restart operation is executed before verifying if the user has the required administrative privileges.

```json
{
  "vulnerable": true,
  "cwe": "CWE-862",
  "severity": "High",
  "reviewed_region": "def restartMySQL(request):\n    try:\n        userID = request.session['userID']\n        finalData = mysqlUtilities.restartMySQL()\n        currentACL = ACLManager.loadedACL(userID)\n\n        if currentACL['admin'] == 1:\n            pass\n        else:\n            return ACLManager.loadErrorJson('FilemanagerAdmin', 0)\n\n        data = {}\n\n        data['status'] = finalData[0]\n        data['error_message'] = finalData[1]\n        json_data = json.dumps(data)\n        return HttpResponse(json_data)\n\n    except KeyError:\n        return redirect(loadLoginPage)",
  "vulnerable_region": "        finalData = mysqlUtilities.restartMySQL()\n        currentACL = ACLManager.loadedACL(userID)\n\n        if currentACL['admin'] == 1:\n            pass\n        else:\n            return ACLManager.loadErrorJson('FilemanagerAdmin', 0)",
  "root_cause": "The authorization check (`currentACL['admin'] == 1`) is performed after the sensitive action (`mysqlUtilities.restartMySQL()`) has already been executed.",
  "exploitability": "High. An attacker simply needs to send a request to the `restartMySQL` endpoint with any valid session.",
  "attack_preconditions": "The attacker must be authenticated to have a valid `userID` in their session.",
  "impact": "Denial of Service (DoS) and disruption of services. Any low-privileged user can restart the MySQL service, potentially causing downtime or interrupting database operations.",
  "non_vulnerable_reason": "",
  "security_invariants": "",
  "invariant_verification": "",
  "residual_risk_notes": "",
  "call_path": "restartMySQL (databases/views.py) \u2192 mysqlUtilities.restartMySQL (plogical/mysqlUtilities.py) \u2192 ACLManager.loadedACL (plogical/acl.py) \u2192 ACLManager.loadErrorJson (plogical/acl.py)"
}
```


System:
You are a senior application security auditor reviewing code for potential vulnerabilities.

You will be given a candidate function, repository and file metadata, a reduced call path, and resolved cross-file context such as helper functions, structs, macros, constants, and related definitions. Use the supplied context actively, but stay strictly grounded in what is actually shown.

Work from the supplied code and surrounding context as the primary evidence. Do not assume a bug exists just because the sample was retrieved.

Analyze both control flow and data flow. Trace the relevant execution paths and track security-relevant data through inputs, transformations, conditions, helper calls, outputs, and side effects.

Reason about attacker influence, identify relevant guards or missing guards, and explain exploitability or non-exploitability before reaching a verdict.

When the code is vulnerable, explain the vulnerable behavior, root cause, realistic attack preconditions, likely impact, and the most precise vulnerability classification justified by the supplied code.

When the code is not vulnerable, explain the concrete invariants and guards that prevent the suspected issue, and note any residual uncertainty that remains from the supplied context.

Keep the analysis concrete and code-grounded. Separate observed evidence from inferred conclusion.

Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: github.com/frappe/lms
Language: Python
File: lms/job/doctype/job_opportunity/job_opportunity.py
Function: validate_urls

Call path: JobOpportunity.validate (lms/job/doctype/job_opportunity/job_opportunity.py) → JobOpportunity.validate_urls (lms/job/doctype/job_opportunity/job_opportunity.py) → frappe.utils.validate_url (frappe.utils)

### Primary Function

```python
def validate_urls(self):
        validate_url(self.company_website, True, ["http", "https"])
```

### Cross-File Context

[JobOpportunity — class — lms/job/doctype/job_opportunity/job_opportunity.py:13]
class JobOpportunity(Document):

[JobOpportunity.validate_urls — caller — lms/job/doctype/job_opportunity/job_opportunity.py:18-19]
def validate_urls(self): validate_url(self.company_website, True, ["http", "https"])

[frappe.utils.validate_url — callee — frappe.utils]
```python
def validate_url(url, is_required=False, valid_prefixes=None):
```

[LMSAssignmentSubmission.validate_url — method — lms/lms/doctype/lms_assignment_submission/lms_assignment_submission.py:32-34]
def validate_url(self): if self.type == "URL" and not validate_url(self.answer, True, ["http", "https"]): frappe.throw(_("Please enter a valid URL."))

[LMSSettings.validate_contact_us_details — method — lms/lms/doctype/lms_settings/lms_settings.py:49-53]
def validate_contact_us_details(self): if self.contact_us_email and not validate_email_address(self.contact_us_email): frappe.throw(_("Please enter a valid Contact Us Email.")) if self.contact_us_url and not validate_url(self.contact_us_url, True, ["http", "https"]): frappe.throw(_("Please enter a valid Contact Us URL."))

Provide the final verdict as strict JSON.

Return only one JSON object and no surrounding prose.

Return one JSON object with these keys:
- vulnerable
- cwe
- severity
- reviewed_region
- vulnerable_region
- root_cause
- exploitability
- attack_preconditions
- impact
- non_vulnerable_reason
- security_invariants
- invariant_verification
- residual_risk_notes
- call_path

Control flow: The function validate_urls has a single linear execution path: it invokes frappe.utils.validate_url with three arguments (self.company_website, True, and a list of allowed prefixes ["http", "https"]). There are no conditional branches, loops, or error-handling constructs in this function. The function does not catch or re-raise exceptions; if validate_url raises an exception (e.g., due to validation failure), it propagates directly to the caller (JobOpportunity.validate).

Data flow: The input is self.company_website, a field on the JobOpportunity document instance. This value is passed unmodified to frappe.utils.validate_url as the url parameter. The second argument True sets is_required, indicating the field must be present. The third argument ["http", "https"] restricts acceptable URL schemes. The validation result (or exception) is handled entirely within the callee; this function does not inspect, transform, or act on the return value.

Verdict: Not Vulnerable

Reviewed region:
[lms/job/doctype/job_opportunity/job_opportunity.py]
```python
def validate_urls(self):
        validate_url(self.company_website, True, ["http", "https"])
```

Why it is not vulnerable: The function delegates URL scheme validation to frappe.utils.validate_url with explicit scheme constraints ["http", "https"] and is_required=True. The valid_prefixes parameter is passed directly to the callee, which is responsible for enforcing that the URL scheme strictly matches one of the allowed prefixes. The invocation correctly restricts schemes to http and https only. Consistent patterns are observed in other parts of the codebase (LMSAssignmentSubmission.validate_url and LMSSettings.validate_contact_us_details) using the same validate_url call with identical prefix constraints, indicating an established validation pattern. While the internal implementation of validate_url is not provided in the supplied context, the current function correctly and completely delegates to it with the appropriate security constraints.

Security invariants:
- The company_website value is validated for scheme compliance before any downstream use.
- Only "http" and "https" schemes are permitted via the valid_prefixes=["http", "https"] argument.
- The is_required=True flag ensures the field is not empty or None.
- The function does not perform any manipulation on the URL value before passing it to the validator, preserving the input integrity for validation.
- Error handling is delegated to the validate_url callee, which is expected to raise an exception on validation failure.

Invariant verification:
- URL scheme restriction to http and https only: holds=true. Evidence: validate_url(self.company_website, True, ["http", "https"]) passes valid_prefixes=["http", "https"] to the callee
- Non-empty URL requirement: holds=true. Evidence: is_required=True is passed as the second argument to validate_url
- Input integrity preserved for validation: holds=true. Evidence: self.company_website is passed directly to validate_url without any intermediate transformation or string manipulation
- Validation enforcement via exception propagation: holds=true. Evidence: The function does not catch exceptions from validate_url, so validation failures will propagate to JobOpportunity.validate and halt document save

Residual risk notes: The security effectiveness of this validation depends entirely on the internal implementation of frappe.utils.validate_url, which is not provided in the supplied cross-file context. Specifically, it cannot be verified from the supplied code whether validate_url performs strict scheme-only validation (without including delimiters like ':' or '//' in the prefix match) or whether it is susceptible to scheme bypass techniques via nested prefix matching. If the callee does not enforce strict scheme boundary checking, the valid_prefixes parameter may not provide adequate protection. This is a dependency-level concern that requires inspecting the validate_url implementation to fully resolve.

No concrete vulnerability impact is demonstrated in the supplied code and context.

An analysis of the provided code reveals a validation bypass vulnerability due to an unchecked return value.

### Vulnerability Analysis
In the `validate_urls` function within the `JobOpportunity` class, the `frappe.utils.validate_url` function is called to ensure that `self.company_website` has a valid structure and uses an acceptable protocol scheme (`http` or `https`):

```python
def validate_urls(self):
    validate_url(self.company_website, True, ["http", "https"])
```

However, cross-file context reveals that `frappe.utils.validate_url` does not raise an exception when validation fails; instead, it returns a boolean value (`True` or `False`). This is clearly demonstrated in other classes, such as `LMSSettings`:

```python
if self.contact_us_url and not validate_url(self.contact_us_url, True, ["http", "https"]):
    frappe.throw(_("Please enter a valid Contact Us URL."))
```

Because `JobOpportunity.validate_urls` entirely ignores the return value of `validate_url`, the validation check has no effect. Any invalid or malicious URL provided in the `company_website` field will be accepted and saved to the database.

### Impact and Exploitability
Since the URL scheme restriction (`["http", "https"]`) is bypassed, an attacker can input malicious URI schemes such as `javascript:alert(1)`. If this URL is later rendered as a clickable link within the application UI (e.g., `<a href="{{ company_website }}">Company</a>`), it will result in a Stored Cross-Site Scripting (XSS) vulnerability. An attacker could exploit this by creating a Job Opportunity document with a malicious payload, leading to script execution in the context of any user viewing the job posting.

### Conclusion
**Verdict:** Vulnerable.
**Root Cause:** Unchecked Return Value (CWE-252), which leads to Improper Input Validation (CWE-20). The return value of the validation function is ignored, nullifying the security check.
**Remediation:** Check the return value of `validate_url` and throw an appropriate exception if validation fails.

```python
def validate_urls(self):
    if not validate_url(self.company_website, True, ["http", "https"]):
        frappe.throw(_("Please enter a valid Company Website URL."))
```

```json
{
  "vulnerable": true,
  "cwe": "CWE-252",
  "severity": "High",
  "reviewed_region": "def validate_urls(self):\n        validate_url(self.company_website, True, [\"http\", \"https\"])",
  "vulnerable_region": "def validate_urls(self):\n        validate_url(self.company_website, True, [\"http\", \"https\"])",
  "root_cause": "The return value of the validate_url function is ignored. Since validate_url returns a boolean instead of raising an exception on failure, ignoring its return value effectively bypasses the validation.",
  "exploitability": "An attacker can provide a URL with an invalid or malicious scheme (such as javascript:) in the company_website field. The application will accept the input despite the validate_url call.",
  "attack_preconditions": "The attacker must have permission to create or update a JobOpportunity document. The application must later render the company_website field in the UI as a clickable link without additional sanitization.",
  "impact": "Stored Cross-Site Scripting (XSS). If an attacker provides a javascript: URL, they can execute arbitrary JavaScript in the context of any user (including administrators) viewing the Job Opportunity.",
  "non_vulnerable_reason": "",
  "security_invariants": "",
  "invariant_verification": "",
  "residual_risk_notes": "",
  "call_path": [
    "JobOpportunity.validate (lms/job/doctype/job_opportunity/job_opportunity.py)",
    "JobOpportunity.validate_urls (lms/job/doctype/job_opportunity/job_opportunity.py)",
    "frappe.utils.validate_url (frappe.utils)"
  ]
}
```


Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: git.kernel.org/pub/scm/linux/kernel/git/stable/linux
Language: C
File: net/sched/sch_ets.c
Function: ets_qdisc_change

Call path: tc_modify_qdisc (net/sched/cls_api.c) → ets_qdisc_change (net/sched/sch_ets.c)

### Primary Function

```c
static int ets_qdisc_change(struct Qdisc *sch, struct nlattr *opt,
				    struct netlink_ext_ack *extack)
{
	unsigned int quanta[TCQ_ETS_MAX_BANDS] = {0};
	struct Qdisc *queues[TCQ_ETS_MAX_BANDS];
	struct ets_sched *q = qdisc_priv(sch);
	struct nlattr *tb[TCA_ETS_MAX + 1];
	unsigned int oldbands = q->nbands;
	u8 priomap[TC_PRIO_MAX + 1];
	unsigned int nstrict = 0;
	unsigned int nbands;
	unsigned int i;
	int err;

	if (!opt) {
		NL_SET_ERR_MSG(extack, "ETS options are required for this operation");
		return -EINVAL;
	}

	err = nla_parse_nested(tb, TCA_ETS_MAX, opt, ets_policy, extack);
	if (err < 0)
		return err;

	if (!tb[TCA_ETS_NBANDS]) {
		NL_SET_ERR_MSG_MOD(extack, "Number of bands is a required argument");
		return -EINVAL;
	}
	nbands = nla_get_u8(tb[TCA_ETS_NBANDS]);
	if (nbands < 1 || nbands > TCQ_ETS_MAX_BANDS) {
		NL_SET_ERR_MSG_MOD(extack, "Invalid number of bands");
		return -EINVAL;
	}
	/* Unless overridden, traffic goes to the last band. */
	memset(priomap, nbands - 1, sizeof(priomap));

	if (tb[TCA_ETS_NSTRICT]) {
		nstrict = nla_get_u8(tb[TCA_ETS_NSTRICT]);
		if (nstrict > nbands) {
			NL_SET_ERR_MSG_MOD(extack, "Invalid number of strict bands");
			return -EINVAL;
		}
	}

	if (tb[TCA_ETS_PRIOMAP]) {
		err = ets_qdisc_priomap_parse(tb[TCA_ETS_PRIOMAP],
					      nbands, priomap, extack);
		if (err)
			return err;
	}

	if (tb[TCA_ETS_QUANTA]) {
		err = ets_qdisc_quanta_parse(sch, tb[TCA_ETS_QUANTA],
				     nbands, nstrict, quanta, extack);
		if (err)
			return err;
	}
	/* If there are more bands than strict + quanta provided, the remaining
	 * ones are ETS with quantum of MTU. Initialize the missing values here.
	 */
	for (i = nstrict; i < nbands; i++) {
		if (!quanta[i])
			quanta[i] = psched_mtu(qdisc_dev(sch));
	}

	/* Before commit, make sure we can allocate all new qdiscs */
	for (i = oldbands; i < nbands; i++) {
		queues[i] = qdisc_create_dflt(sch->dev_queue, &pfifo_qdisc_ops,
					      ets_class_id(sch, &q->classes[i]),
					      extack);
		if (!queues[i]) {
			while (i > oldbands)
				qdisc_put(queues[--i]);
			return -ENOMEM;
		}
	}

	sch_tree_lock(sch);

	for (i = nbands; i < oldbands; i++) {
		if (cl_is_active(&q->classes[i]))
			list_del_init(&q->classes[i].alist);
		qdisc_purge_queue(q->classes[i].qdisc);
	}

	WRITE_ONCE(q->nbands, nbands);
	for (i = nstrict; i < q->nstrict; i++) {
		if (q->classes[i].qdisc->q.qlen) {
			list_add_tail(&q->classes[i].alist, &q->active);
			q->classes[i].deficit = quanta[i];
		}
	}
	for (i = q->nstrict; i < nstrict; i++) {
		if (cl_is_active(&q->classes[i]))
			list_del_init(&q->classes[i].alist);
	}
	WRITE_ONCE(q->nstrict, nstrict);
	memcpy(q->prio2band, priomap, sizeof(priomap));

	for (i = 0; i < q->nbands; i++)
		WRITE_ONCE(q->classes[i].quantum, quanta[i]);

	for (i = oldbands; i < q->nbands; i++) {
		q->classes[i].qdisc = queues[i];
		if (q->classes[i].qdisc != &noop_qdisc)
			qdisc_hash_add(q->classes[i].qdisc, true);
	}

	sch_tree_unlock(sch);

	ets_offload_change(sch);
	for (i = q->nbands; i < oldbands; i++) {
		qdisc_put(q->classes[i].qdisc);
		q->classes[i].qdisc = NULL;
		WRITE_ONCE(q->classes[i].quantum, 0);
		q->classes[i].deficit = 0;
		memset(&q->classes[i].bstats, 0, sizeof(q->classes[i].bstats));
		memset(&q->classes[i].qstats, 0, sizeof(q->classes[i].qstats));
	}
	return 0;
}
```

### Cross-File Context

[cl_is_active — function — net/sched/sch_ets.c:77]
```c
static bool cl_is_active(struct ets_class *cl)
{
	return !list_empty(&cl->alist);
}
```

[ets_sched — struct — net/sched/sch_ets.c:49]
```c
struct ets_sched {
	struct list_head active;
	struct tcf_proto __rcu *filter_list;
	struct tcf_block *block;
	unsigned int nbands;
	unsigned int nstrict;
	u8 prio2band[TC_PRIO_MAX + 1];
	struct ets_class classes[TCQ_ETS_MAX_BANDS];
};
```

[ets_class — struct — net/sched/sch_ets.c:40]
```c
struct ets_class {
	struct list_head alist;
	struct Qdisc *qdisc;
	u32 quantum;
	u32 deficit;
	struct gnet_stats_basic_packed bstats;
	struct gnet_stats_queue qstats;
};
```

[TCQ_ETS_MAX_BANDS — constant — include/uapi/linux/pkt_sched.h]
TCQ_ETS_MAX_BANDS → 16  (include/uapi/linux/pkt_sched.h)

[list_del_init — macro — include/linux/list.h]
list_del_init → #define list_del_init(list) do { __list_del(list); INITIALIZE_LIST_HEAD(list); } while (0)  (include/linux/list.h)

[list_add_tail — macro — include/linux/list.h]
list_add_tail → #define list_add_tail(new, head) __list_add(new, head->prev, head)  (include/linux/list.h)

Control flow: The function handles ETS qdisc reconfiguration through several sequential stages: (1) parameter parsing and validation via nla_parse_nested, (2) nbands validation (1-16 range), (3) nstrict validation (must be <= nbands), (4) optional priomap and quanta parsing with error propagation, (5) quanta initialization for bands lacking explicit values, (6) allocation of new qdiscs for bands in [oldbands, nbands) with cleanup on failure, (7) locking with sch_tree_lock, (8) removal of classes being eliminated when nbands decreases (bands [nbands, oldbands) removed from active list, purged), (9) WRITE_ONCE update of nbands, (10) bands transitioning from strict to ETS (range [nstrict, q->nstrict)) are added to active list with deficit initialization, (11) bands transitioning from ETS to strict (range [q->nstrict, nstrict)) are removed from active list, (12) WRITE_ONCE update of nstrict and priomap copy, (13) quanta update for all active bands, (14) hash table registration for newly allocated qdiscs, (15) unlock, (16) offload notification, (17) final cleanup of eliminated class entries.

Data flow: User-supplied nlattr options flow through nla_parse_nested into tb[] array. nbands (u8) and nstrict (u8) are extracted and bounded-checked. priomap is initialized to default value (nbands-1) and optionally overridden by ets_qdisc_priomap_parse. quanta array is populated from user input via ets_qdisc_quanta_parse, then any zero entries in ETS bands are filled with psched_mtu. These validated parameters are written into the ets_sched structure under lock protection: nbands via WRITE_ONCE, nstrict via WRITE_ONCE, prio2band via memcpy, and per-class quantum via WRITE_ONCE. List operations on the active list use class list nodes (&q->classes[i].alist).

Verdict: Not Vulnerable

Reviewed region:
[net/sched/sch_ets.c]
```c
static int ets_qdisc_change(struct Qdisc *sch, struct nlattr *opt,
				    struct netlink_ext_ack *extack)
{
	unsigned int quanta[TCQ_ETS_MAX_BANDS] = {0};
	struct Qdisc *queues[TCQ_ETS_MAX_BANDS];
	struct ets_sched *q = qdisc_priv(sch);
	struct nlattr *tb[TCA_ETS_MAX + 1];
	unsigned int oldbands = q->nbands;
	u8 priomap[TC_PRIO_MAX + 1];
	unsigned int nstrict = 0;
	unsigned int nbands;
	unsigned int i;
	int err;

	if (!opt) {
		NL_SET_ERR_MSG(extack, "ETS options are required for this operation");
		return -EINVAL;
	}

	err = nla_parse_nested(tb, TCA_ETS_MAX, opt, ets_policy, extack);
	if (err < 0)
		return err;

	if (!tb[TCA_ETS_NBANDS]) {
		NL_SET_ERR_MSG_MOD(extack, "Number of bands is a required argument");
		return -EINVAL;
	}
	nbands = nla_get_u8(tb[TCA_ETS_NBANDS]);
	if (nbands < 1 || nbands > TCQ_ETS_MAX_BANDS) {
		NL_SET_ERR_MSG_MOD(extack, "Invalid number of bands");
		return -EINVAL;
	}
	/* Unless overridden, traffic goes to the last band. */
	memset(priomap, nbands - 1, sizeof(priomap));

	if (tb[TCA_ETS_NSTRICT]) {
		nstrict = nla_get_u8(tb[TCA_ETS_NSTRICT]);
		if (nstrict > nbands) {
			NL_SET_ERR_MSG_MOD(extack, "Invalid number of strict bands");
			return -EINVAL;
		}
	}

	if (tb[TCA_ETS_PRIOMAP]) {
		err = ets_qdisc_priomap_parse(tb[TCA_ETS_PRIOMAP],
					      nbands, priomap, extack);
		if (err)
			return err;
	}

	if (tb[TCA_ETS_QUANTA]) {
		err = ets_qdisc_quanta_parse(sch, tb[TCA_ETS_QUANTA],
				     nbands, nstrict, quanta, extack);
		if (err)
			return err;
	}
	/* If there are more bands than strict + quanta provided, the remaining
	 * ones are ETS with quantum of MTU. Initialize the missing values here.
	 */
	for (i = nstrict; i < nbands; i++) {
		if (!quanta[i])
			quanta[i] = psched_mtu(qdisc_dev(sch));
	}

	/* Before commit, make sure we can allocate all new qdiscs */
	for (i = oldbands; i < nbands; i++) {
		queues[i] = qdisc_create_dflt(sch->dev_queue, &pfifo_qdisc_ops,
					      ets_class_id(sch, &q->classes[i]),
					      extack);
		if (!queues[i]) {
			while (i > oldbands)
				qdisc_put(queues[--i]);
			return -ENOMEM;
		}
	}

	sch_tree_lock(sch);

	for (i = nbands; i < oldbands; i++) {
		if (cl_is_active(&q->classes[i]))
			list_del_init(&q->classes[i].alist);
		qdisc_purge_queue(q->classes[i].qdisc);
	}

	WRITE_ONCE(q->nbands, nbands);
	for (i = nstrict; i < q->nstrict; i++) {
		if (q->classes[i].qdisc->q.qlen) {
			list_add_tail(&q->classes[i].alist, &q->active);
			q->classes[i].deficit = quanta[i];
		}
	}
	for (i = q->nstrict; i < nstrict; i++) {
		if (cl_is_active(&q->classes[i]))
			list_del_init(&q->classes[i].alist);
	}
	WRITE_ONCE(q->nstrict, nstrict);
	memcpy(q->prio2band, priomap, sizeof(priomap));

	for (i = 0; i < q->nbands; i++)
		WRITE_ONCE(q->classes[i].quantum, quanta[i]);

	for (i = oldbands; i < q->nbands; i++) {
		q->classes[i].qdisc = queues[i];
		if (q->classes[i].qdisc != &noop_qdisc)
			qdisc_hash_add(q->classes[i].qdisc, true);
	}

	sch_tree_unlock(sch);

	ets_offload_change(sch);
	for (i = q->nbands; i < oldbands; i++) {
		qdisc_put(q->classes[i].qdisc);
		q->classes[i].qdisc = NULL;
		WRITE_ONCE(q->classes[i].quantum, 0);
		q->classes[i].deficit = 0;
		memset(&q->classes[i].bstats, 0, sizeof(q->classes[i].bstats));
		memset(&q->classes[i].qstats, 0, sizeof(q->classes[i].qstats));
	}
	return 0;
}
```

Why it is not vulnerable: Linked list nodes are never re-added while still present in the active list. The list_add_tail operation (adding bands transitioning from strict to ETS in range [nstrict, q->nstrict)) and the list_del_init operations (removing bands being eliminated in range [nbands, oldbands) and removing bands transitioning from ETS to strict in range [q->nstrict, nstrict)) operate on non-overlapping index ranges. Since nstrict <= nbands <= oldbands holds due to validation and the removal loop executes before the addition loop, no class node can be both deleted and re-added in the same call. Additionally, strict bands by design are not members of the active list (used for deficit round-robin scheduling of ETS bands), so bands transitioning from strict to ETS are adding fresh list nodes that were never in the active list. Bands transitioning from ETS to strict are properly removed before the transition, preventing stale references.

Security invariants:
- List node removal precedes re-addition: The removal loop for eliminated bands (range [nbands, oldbands)) executes before the addition loop for newly ETS bands (range [nstrict, q->nstrict)), and these ranges are provably disjoint because nstrict <= nbands <= oldbands.
- Strict-to-ETS transition uses clean list nodes: Bands in [nstrict, q->nstrict) that transition from strict to ETS were previously operating in strict mode, which does not use the active list, so their list nodes are not present in the active list when added.
- ETS-to-strict transition removes nodes before transition: Bands in [q->nstrict, nstrict) transitioning from ETS to strict are explicitly removed from the active list via list_del_init, preventing stale entries.
- Band count validation prevents out-of-bounds: nbands is validated to be in [1, TCQ_ETS_MAX_BANDS] (1-16), and nstrict is validated to be <= nbands, ensuring all list operations index valid class entries.
- Lock-protected list operations: sch_tree_lock/sch_tree_unlock bracket all active list modifications, preventing concurrent corruption.

Invariant verification:
- Non-overlapping removal and addition index ranges: holds=true. Evidence: Removal loop iterates i from nbands to oldbands (exclusive), while addition loop iterates i from nstrict to q->nstrict (exclusive). Since nstrict <= nbands is enforced by validation, and nbands <= oldbands for the removal loop to execute, the ranges [nbands, oldbands) and [nstrict, q->nstrict) are disjoint. The removal loop executes at lines before the addition loop (ordering within locked section).
- Strict bands not present in active list before transition to ETS: holds=true. Evidence: Bands 0 through nstrict-1 are strict bands that use direct scheduling, not deficit round-robin. The active list is exclusively for ETS bands (nstrict through nbands-1). When nstrict decreases, bands in [nstrict, q->nstrict) transition from strict to ETS, and their list nodes were never in the active list. This is verified by the invariant that cl_is_active only returns true for bands previously added via list_add_tail in the ETS range.
- ETS-to-strict transition removes list nodes: holds=true. Evidence: The loop 'for (i = q->nstrict; i < nstrict; i++) { if (cl_is_active(&q->classes[i])) list_del_init(&q->classes[i].alist); }' explicitly removes any active list nodes for bands transitioning from ETS to strict, using list_del_init to both remove and re-initialize the list node.
- nstrict bounded by nbands: holds=true. Evidence: Validation at 'if (nstrict > nbands) { NL_SET_ERR_MSG_MOD(extack, "Invalid number of strict bands"); return -EINVAL; }' ensures nstrict <= nbands, which is critical for the disjointness of the list operation ranges.
- Mutex protection for list modifications: holds=true. Evidence: sch_tree_lock(sch) is called before all list operations (list_del_init, list_add_tail, and cl_is_active checks), and sch_tree_unlock(sch) is called after all modifications, ensuring atomicity.

Residual risk notes: The safety of the active list operations depends on the invariant that strict bands (indices 0 to nstrict-1) are never members of the active list during normal operation. This invariant is maintained by the deficit round-robin dequeuing logic in ets_dequeue (not shown in context). If that logic were to incorrectly add or fail to remove list nodes for strict-band transitions, duplicate entries could theoretically arise. However, based on the current code structure, the transition logic in ets_qdisc_change correctly handles all four scenarios: bands eliminated (removed and purged), bands added (new allocation), strict-to-ETS (added fresh), and ETS-to-strict (removed before transition).

Based on a security analysis of the provided code and context, the `ets_qdisc_change` function correctly implements changes to the ETS qdisc configuration without exhibiting obvious vulnerabilities such as use-after-free, out-of-bounds access, or list corruption.

Here is a summary of the checks validating its safety:
1. **Input Validation**: The number of bands (`nbands`) and strict bands (`nstrict`) are properly validated. It ensures `nbands` is between 1 and `TCQ_ETS_MAX_BANDS` and `nstrict <= nbands`.
2. **Safe Allocations**: Before mutating the active qdisc state, the function safely allocates all new queues. If `qdisc_create_dflt` fails, it rolls back correctly by `qdisc_put`ting only the newly allocated items.
3. **Synchronization**: State changes to the queue structures, active class lists, and active variables are properly protected under `sch_tree_lock(sch)`.
4. **List Safety**: When elements transition between strict and non-strict (ETS) bands, they are added to or removed from `q->active`. `list_add_tail` is correctly used only for bands that have packets (`qlen > 0`) and transitioning bands correctly reflect their active/inactive status via `list_del_init()`.
5. **No Use-After-Free**: Unused queues are securely removed from the hierarchy and purged before dropping `sch_tree_lock(sch)`. Destroying `qdisc` references happens at the very end of the function ensuring no concurrent queue operation triggers a use-after-free.

No specific security issue was identified in this snippet.

```json
{
  "vulnerable": false,
  "cwe": "N/A",
  "severity": "None",
  "reviewed_region": "ets_qdisc_change",
  "vulnerable_region": "N/A",
  "root_cause": "N/A",
  "exploitability": "N/A",
  "attack_preconditions": "N/A",
  "impact": "N/A",
  "non_vulnerable_reason": "The function correctly validates all inputs (e.g., verifying nbands is within [1, TCQ_ETS_MAX_BANDS] and nstrict <= nbands). It securely manages qdisc replacements by fully allocating new queues before committing changes, effectively preventing memory corruption or inconsistent state in error cases (via graceful rollback). The active band states are properly synced utilizing list primitives (list_add_tail, list_del_init) while guarded by the sch_tree_lock, mitigating list corruption or race conditions. Queues removed from the hierarchy are safely purged and freed at the end of the operation, ensuring no use-after-free vulnerabilities occur.",
  "security_invariants": [
    "Input boundaries are strictly enforced before state modifications (nbands <= TCQ_ETS_MAX_BANDS, nstrict <= nbands).",
    "List manipulations on `q->active` for transitioning strict/non-strict bands are guarded by `sch_tree_lock()`.",
    "Bands scheduled for deletion are purged and their active status updated before `q->nstrict` or `q->nbands` are modified."
  ],
  "invariant_verification": [
    "Input validation check: `if (nbands < 1 || nbands > TCQ_ETS_MAX_BANDS)` and `if (nstrict > nbands)` return early.",
    "State lock: The core state transition is enclosed in `sch_tree_lock(sch)` and `sch_tree_unlock(sch)`.",
    "qdisc cleanup: Outdated queues are collected and passed to `qdisc_put()` after unlocking, preventing concurrent UAF."
  ],
  "residual_risk_notes": "No immediate residual risk identified in the provided context; concurrent datapath access to the lists is safely serialized by the qdisc tree lock.",
  "call_path": "tc_modify_qdisc (net/sched/cls_api.c) -> ets_qdisc_change (net/sched/sch_ets.c)"
}
```


Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: github.com/getgrav/grav.git
Language: PHP
File: system/src/Grav/Common/Utils.php
Function: isDangerousFunction

Call path: Grav\Common\Twig\Extension\GravExtension::filterFilter (system/src/Grav/Common/Twig/Extension/GravExtension.php) → Grav\Common\Twig\Extension\GravExtension::mapFilter (system/src/Grav/Common/Twig/Extension/GravExtension.php) → Grav\Common\Twig\Twig::registerUndefinedFunctionCallback (system/src/Grav/Common/Twig/Twig.php) → Grav\Common\Twig\Twig::registerUndefinedFilterCallback (system/src/Grav/Common/Twig/Twig.php) → Grav\Common\Utils::isDangerousFunction (system/src/Grav/Common/Utils.php) → Grav\Common\Utils::isFilesystemFunction (system/src/Grav/Common/Utils.php)

### Primary Function

```php
public static function isDangerousFunction($name): bool
{
    static $commandExecutionFunctions = [
        'exec',
        'passthru',
        'system',
        'shell_exec',
        'popen',
        'proc_open',
        'pcntl_exec',
    ];

    static $codeExecutionFunctions = [
        'assert',
        'preg_replace',
        'create_function',
        'include',
        'include_once',
        'require',
        'require_once'
    ];

    static $callbackFunctions = [
        'ob_start' => 0,
        'array_diff_uassoc' => -1,
        'array_diff_ukey' => -1,
        'array_filter' => 1,
        'array_intersect_uassoc' => -1,
        'array_intersect_ukey' => -1,
        'array_map' => 0,
        'array_reduce' => 1,
        'array_udiff_assoc' => -1,
        'array_udiff_uassoc' => [-1, -2],
        'array_udiff' => -1,
        'array_uintersect_assoc' => -1,
        'array_uintersect_uassoc' => [-1, -2],
        'array_uintersect' => -1,
        'array_walk_recursive' => 1,
        'array_walk' => 1,
        'assert_options' => 1,
        'uasort' => 1,
        'uksort' => 1,
        'usort' => 1,
        'preg_replace_callback' => 1,
        'spl_autoload_register' => 0,
        'iterator_apply' => 1,
        'call_user_func' => 0,
        'call_user_func_array' => 0,
        'register_shutdown_function' => 0,
        'register_tick_function' => 0,
        'set_error_handler' => 0,
        'set_exception_handler' => 0,
        'session_set_save_handler' => [0, 1, 2, 3, 4, 5],
        'sqlite_create_aggregate' => [2, 3],
        'sqlite_create_function' => 2,
    ];

    static $informationDiscosureFunctions = [
        'phpinfo',
        'posix_mkfifo',
        'posix_getlogin',
        'posix_ttyname',
        'getenv',
        'get_current_user',
        'proc_get_status',
        'get_cfg_var',
        'disk_free_space',
        'disk_total_space',
        'diskfreespace',
        'getcwd',
        'getlastmo',
        'getmygid',
        'getmyinode',
        'getmypid',
        'getmyuid'
    ];

    static $otherFunctions = [
        'extract',
        'parse_str',
        'putenv',
        'ini_set',
        'mail',
        'header',
        'proc_nice',
        'proc_terminate',
        'proc_close',
        'pfsockopen',
        'fsockopen',
        'apache_child_terminate',
        'posix_kill',
        'posix_mkfifo',
        'posix_setpgid',
        'posix_setsid',
        'posix_setuid',
    ];

    if (is_array($name) || strpos($name, ":") !== false) {
        return false;
    }

    if (in_array($name, $commandExecutionFunctions)) {
        return true;
    }

    if (in_array($name, $codeExecutionFunctions)) {
        return true;
    }

    if (isset($callbackFunctions[$name])) {
        return true;
    }

    if (in_array($name, $informationDiscosureFunctions)) {
        return true;
    }

    if (in_array($name, $otherFunctions)) {
        return true;
    }

    return static::isFilesystemFunction($name);
}
```

### Cross-File Context

[Grav\Common\Utils — class — system/src/Grav/Common/Utils.php:46]
abstract class Utils

[Grav\Common\Utils::isDangerousFunction — sink — system/src/Grav/Common/Utils.php:1956-2078]
public static function isDangerousFunction($name): bool { static $commandExecutionFunctions = [ 'exec', 'passthru', 'system', 'shell_exec', 'popen', 'proc_open', 'pcntl_exec', ]; static $codeExecutionFunctions = [ 'assert', 'preg_replace', 'create_function', 'include', 'include_once', 'require', 'require_once' ]; static $callbackFunctions = [ 'ob_start' => 0, 'array_diff_uassoc' => -1, 'array_diff_ukey' => -1, 'array_filter' => 1, 'array_intersect_uassoc' => -1, 'array_intersect_ukey' => -1, 'array_map' => 0, 'array_reduce' => 1, 'array_udiff_assoc' => -1, 'array_udiff_uassoc' => [-1, -2], 'array_udiff' => -1, 'array_uintersect_assoc' => -1, 'array_uintersect_uassoc' => [-1, -2], 'array_uintersect' => -1, 'array_walk_recursive' => 1, 'array_walk' => 1, 'assert_options' => 1, 'uasort' => 1, 'uksort' => 1, 'usort' => 1, 'preg_replace_callback' => 1, 'spl_autoload_register' => 0, 'iterator_apply' => 1, 'call_user_func' => 0, 'call_user_func_array' => 0, 'register_shutdown_function' => 0, 'register_tick_function' => 0, 'set_error_handler' => 0, 'set_exception_handler' => 0, 'session_set_save_handler' => [0, 1, 2, 3, 4, 5], 'sqlite_create_aggregate' => [2, 3], 'sqlite_create_function' => 2, ]; static $informationDiscosureFunctions = [ 'phpinfo', 'posix_mkfifo', 'posix_getlogin', 'posix_ttyname', 'getenv', 'get_current_user', 'proc_get_status', 'get_cfg_var', 'disk_free_space', 'disk_total_space', 'diskfreespace', 'getcwd', 'getlastmo', 'getmygid', 'getmyinode', 'getmypid', 'getmyuid' ]; static $otherFunctions = [ 'extract', 'parse_str', 'putenv', 'ini_set', 'mail', 'header', 'proc_nice', 'proc_terminate', 'proc_close', 'pfsockopen', 'fsockopen', 'apache_child_terminate', 'posix_kill', 'posix_mkfifo', 'posix_setpgid', 'posix_setsid', 'posix_setuid', ]; if (is_array($name) || strpos($name, ":") !== false) { return false; } if (in_array($name, $commandExecutionFunctions)) { return true; } if (in_array($name, $codeExecutionFunctions)) { return true; } if (isset($callbackFunctions[$name])) { return true; } if (in_array($name, $informationDiscosureFunctions)) { return true; } if (in_array($name, $otherFunctions)) { return true; } return static::isFilesystemFunction($name); }

[Grav\Common\Utils::isFilesystemFunction — callee — system/src/Grav/Common/Utils.php:2084]
public static function isFilesystemFunction(string $name): bool { static $fileWriteFunctions = [ 'fopen', 'tmpfile', 'bzopen', 'gzopen', 'chgrp', 'chmod', 'chown', 'copy', 'file_put_contents', 'lchgrp', 'lchown', 'link', 'mkdir', 'move_uploaded_file', 'rename', 'rmdir', 'symlink', 'tempnam', 'touch', 'unlink', 'imagepng', 'imagewbmp', 'image2wbmp', 'imagejpeg', 'imagexbm', 'imagegif', 'imagegd', 'imagegd2', 'iptcembed', 'ftp_get', 'ftp_nb_get', ]; static $fileContentFunctions = [ 'file_get_contents', 'file', 'filegroup', 'fileinode', 'fileowner', 'fileperms', 'glob', 'is_executable', 'is_uploaded_file', 'parse_ini_file', 'readfile', 'readlink', 'realpath', 'gzfile', 'readgzfile', 'stat', 'imagecreatefromgif', 'imagecreatefromjpeg', 'imagecreatefrompng', 'imagecreatefromwbmp', 'imagecreatefromxbm', 'imagecreatefromxpm', 'ftp_put', 'ftp_nb_put', 'hash_update_file', 'highlight_file', 'show_source', 'php_strip_whitespace', ]; if (in_array($name, $fileWriteFunctions)) { return true; } if (in_array($name, $fileContentFunctions)) { return true; } return false; }

[Grav\Common\Twig\Extension\GravExtension::filterFilter — entry — system/src/Grav/Common/Twig/Extension/GravExtension.php:1709]
function filterFilter(Environment $env, $array, $arrow) { if (!$arrow instanceof \Closure && !is_string($arrow) || Utils::isDangerousFunction($arrow)) { throw new RuntimeError('Twig |filter("' . $arrow . '") is not allowed.'); } return twig_array_filter($env, $array, $arrow); }

[Grav\Common\Twig\Extension\GravExtension::mapFilter — entry — system/src/Grav/Common/Twig/Extension/GravExtension.php:1725]
function mapFilter(Environment $env, $array, $arrow) { if (!$arrow instanceof \Closure && !is_string($arrow) || Utils::isDangerousFunction($arrow)) { throw new RuntimeError('Twig |map("' . $arrow . '") is not allowed.'); } return twig_array_map($env, $array, $arrow); }

[Grav\Common\Twig\Twig — class — system/src/Grav/Common/Twig/Twig.php:49]
class Twig

[Grav\Common\Twig\Extension\GravExtension — class — system/src/Grav/Common/Twig/Extension/GravExtension.php:73]
class GravExtension extends AbstractExtension implements GlobalsInterface

[namespace Grav\Common — namespace — system/src/Grav/Common/Utils.php:11]
namespace Grav\Common;

[namespace Grav\Common\Twig\Extension — namespace — system/src/Grav/Common/Twig/Extension/GravExtension.php:11]
namespace Grav\Common\Twig\Extension;

Control flow: Twig template rendering invokes user-provided expressions. When a |filter or |map Twig filter is used, GravExtension::filterFilter or GravExtension::mapFilter is called with the arrow callable name. These functions call Utils::isDangerousFunction($arrow) to check if the function is dangerous. Similarly, Twig's undefined function/filter callbacks in Twig.php call Utils::isDangerousFunction($name) before allowing undefined PHP functions to be used as Twig functions/filters. If isDangerousFunction returns false, the function is allowed to execute. The pre-fix isDangerousFunction has an incomplete blocklist and lacks case normalization, allowing bypass.

Data flow: User-controlled Twig template content -> arrow callable string (e.g., 'unserialize', 'System') -> GravExtension::filterFilter/mapFilter or Twig undefined callback -> Utils::isDangerousFunction($name) -> incomplete blocklist check without strtolower -> returns false for dangerous but unlisted or case-mangled function names -> PHP function executed as Twig function/filter.

Verdict: Vulnerable
Vulnerable region:
[system/src/Grav/Common/Utils.php]
```php
'extract',
            'parse_str',
            'putenv',
            'ini_set',
            'mail',
            'header',
            'proc_nice',
            'proc_terminate',
            'proc_close',
            'pfsockopen',
            'fsockopen',
            'apache_child_terminate',
            'posix_kill',
            'posix_mkfifo',
            'posix_setpgid',
            'posix_setsid',
            'posix_setuid',
        ];

        if (is_array($name) || strpos($name, ":") !== false) {
            return false;
        }

        if (in_array($name, $commandExecutionFunctions)) {
            return true;
```
CWE: CWE-393,CWE-74
Severity: HIGH

Root cause: Utils::isDangerousFunction() serves as the security gate that decides whether a PHP function can be called from Twig templates. The pre-fix implementation has three weaknesses: (1) the $otherFunctions blocklist is incomplete, missing 'unserialize', 'ini_alter', 'simplexml_load_file', 'simplexml_load_string', 'forward_static_call', and 'forward_static_call_array'; (2) there is no strtolower() normalization before comparing $name against the lowercase blocklist entries, so mixed-case function names like 'System' bypass the check; (3) there is no backslash check, so namespaced references like '\exec' could bypass the blocklist. These gaps allow an attacker with Twig template injection capability to invoke dangerous PHP functions that bypass the SSTI protection.

Exploitability: An attacker with the ability to inject Twig template code (e.g., through page content editing, theme uploads, or admin access) can pass unlisted dangerous function names (e.g., 'unserialize', 'forward_static_call') or use case variations (e.g., 'System', 'EXEC') to bypass Utils::isDangerousFunction(). The function is called from GravExtension::filterFilter, GravExtension::mapFilter, and Twig undefined function/filter callbacks, which are all reachable from user-controlled Twig templates.

Attack preconditions:
- Attacker must be able to inject or control Twig template content (e.g., via page editing, theme file upload, or admin panel access)
- The Grav instance must have Twig template processing enabled (default behavior)
- The system.twig.undefined_functions or system.twig.undefined_filters configuration must allow undefined function/filter usage, or the attacker must use |filter or |map Twig filters

Likely impact: An attacker with the ability to inject Twig template code (e.g., through page content editing, theme uploads, or admin access) can pass unlisted dangerous function names (e.g., 'unserialize', 'forward_static_call') or use case variations (e.g., 'System', 'EXEC') to bypass Utils::isDangerousFunction(). The function is called from GravExtension::filterFilter, GravExtension::mapFilter, and Twig undefined function/filter callbacks, which are all reachable from user-controlled Twig templates.

An analysis of the `isDangerousFunction` method reveals that it uses a **denylist** approach to block specific PHP functions from being executed as callbacks or undefined functions in Twig.

While the denylist covers many obvious command/code execution and filesystem functions, there are two primary weaknesses in this guard that could lead to bypasses:

1. **Denylist Incompleteness:** As with any denylist, it is practically impossible to cover all dangerous PHP functions. Attackers often bypass such lists by finding obscure functions that achieve similar goals (e.g., `mb_ereg_replace_callback`, `imap_open`, or newly added PHP functions).

2. **Explicit Bypasses for Static Methods and Array Callables:**
   ```php
   if (is_array($name) || strpos($name, ":") !== false) {
       return false;
   }
   ```
   The code explicitly returns `false` (meaning "not dangerous") if `$name` is an array or contains a colon (`:`).
   - **Array Callables:** While the provided `mapFilter` and `filterFilter` contexts attempt to guard against non-strings (`!is_string($arrow)` will throw if an array is passed), other callers (such as `registerUndefinedFunctionCallback`) might not have this strict type check. If an array callable like `['SomeClass', 'dangerousMethod']` or `['system']` reaches this function, it is immediately permitted.
   - **Static Method Strings:** Standard PHP callables can be represented as strings containing double colons (e.g., `"ClassName::methodName"`). Since the code checks `strpos($name, ":") !== false`, any string containing a colon completely bypasses the denylist. If an attacker can invoke a Twig filter or undefined function and point it to a gadget—a publicly accessible static method in the application's loaded classes that performs dangerous actions (like instantiating a process or manipulating files)—they can achieve Remote Code Execution (RCE) or other impacts.

**Verdict:** Vulnerable. The reliance on an incomplete denylist and the explicit `return false` bypass for strings containing colons (static method callbacks) and arrays allows attackers to execute arbitrary code or unauthorized actions if they can chain this with an available static gadget or another unprotected caller.

```json
{
  "vulnerable": true,
  "cwe": "CWE-184",
  "severity": "High",
  "reviewed_region": "Grav\\Common\\Utils::isDangerousFunction",
  "vulnerable_region": "if (is_array($name) || strpos($name, \":\") !== false) {\n        return false;\n    }",
  "root_cause": "The denylist function explicitly returns `false` (meaning the function is safe) if the callable name is an array or contains a colon (`:`). This allows attackers to bypass the blocklist by supplying a static class method (e.g., `\"ClassName::method\"`), which circumvents all checks.",
  "exploitability": "Highly exploitable if the attacker has control over Twig templates or template variables that are evaluated by filters like `|filter` or `|map`. By supplying a string like `\"SomeGadgetClass::dangerousStaticMethod\"`, the payload avoids the denylist and is passed to `twig_array_filter` or `twig_array_map`.",
  "attack_preconditions": "The attacker must be able to inject template code or control variables that are used as callable arguments for functions restricted by `isDangerousFunction` (e.g. `filter`, `map`, or undefined function callbacks).",
  "impact": "Remote Code Execution (RCE) or arbitrary file manipulation, depending on the static gadgets available in the loaded PHP classes.",
  "non_vulnerable_reason": "",
  "security_invariants": [],
  "invariant_verification": [],
  "residual_risk_notes": "",
  "call_path": [
    "Grav\\Common\\Twig\\Extension\\GravExtension::filterFilter (system/src/Grav/Common/Twig/Extension/GravExtension.php)",
    "Grav\\Common\\Utils::isDangerousFunction (system/src/Grav/Common/Utils.php)"
  ]
}
```

The denylist function explicitly returns `false` (meaning the function is safe) if the callable name is an array or contains a colon (`:`). This allows attackers to bypass the blocklist by supplying a static class method (e.g., `"ClassName::method"`), which circumvents all checks.

Highly exploitable if the attacker has control over Twig templates or template variables that are evaluated by filters like `|filter` or `|map`. By supplying a string like `"SomeGadgetClass::dangerousStaticMethod"`, the payload avoids the denylist and is passed to `twig_array_filter` or `twig_array_map`.


Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: git.kernel.org/pub/scm/linux/kernel/git/stable/linux
Language: Assembly
File: arch/riscv/kernel/entry.S
Function: handle_exception

Call path: handle_exception (arch/riscv/kernel/entry.S) → trace_hardirqs_off (kernel/trace/trace_preemptirq.c) → trace_hardirqs_on (kernel/trace/trace_preemptirq.c) → __trace_hardirqs_off (arch/riscv/kernel/trace_irq.c) → __trace_hardirqs_on (arch/riscv/kernel/trace_irq.c)

### Primary Function

```assembly
.globl handle_exception
handle_exception:
	csrrw tp, CSR_SCRATCH, tp
	bnez tp, _save_context
_restore_kernel_tpsp:
	csrr tp, CSR_SCRATCH
	REG_S sp, TASK_TI_KERNEL_SP(tp)
_save_context:
	REG_S sp, TASK_TI_USER_SP(tp)
	REG_L sp, TASK_TI_KERNEL_SP(tp)
	addi sp, sp, -(PT_SIZE_ON_STACK)
	REG_S x1,  PT_RA(sp)
	REG_S x3,  PT_GP(sp)
	REG_S x5,  PT_T0(sp)
	REG_S x6,  PT_T1(sp)
	REG_S x7,  PT_T2(sp)
	REG_S x8,  PT_S0(sp)
	REG_S x9,  PT_S1(sp)
	REG_S x10, PT_A0(sp)
	REG_S x11, PT_A1(sp)
	REG_S x12, PT_A2(sp)
	REG_S x13, PT_A3(sp)
	REG_S x14, PT_A4(sp)
	REG_S x15, PT_A5(sp)
	REG_S x16, PT_A6(sp)
	REG_S x17, PT_A7(sp)
	REG_S x18, PT_S2(sp)
	REG_S x19, PT_S3(sp)
	REG_S x20, PT_S4(sp)
	REG_S x21, PT_S5(sp)
	REG_S x22, PT_S6(sp)
	REG_S x23, PT_S7(sp)
	REG_S x24, PT_S8(sp)
	REG_S x25, PT_S9(sp)
	REG_S x26, PT_S10(sp)
	REG_S x27, PT_S11(sp)
	REG_S x28, PT_T3(sp)
	REG_S x29, PT_T4(sp)
	REG_S x30, PT_T5(sp)
	REG_S x31, PT_T6(sp)
	li t0, SR_SUM | SR_FS
	REG_L s0, TASK_TI_USER_SP(tp)
	csrrc s1, CSR_STATUS, t0
	csrr s2, CSR_EPC
	csrr s3, CSR_TVAL
	csrr s4, CSR_CAUSE
	csrr s5, CSR_SCRATCH
	REG_S s0, PT_SP(sp)
	REG_S s1, PT_STATUS(sp)
	REG_S s2, PT_EPC(sp)
	REG_S s3, PT_BADADDR(sp)
	REG_S s4, PT_CAUSE(sp)
	REG_S s5, PT_TP(sp)
	csrw CSR_SCRATCH, x0
.option push
.option norelax
	la gp, __global_pointer$
.option pop
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_off
#endif
#ifdef CONFIG_CONTEXT_TRACKING
	li   a0, SR_PP
	and a0, s1, a0
	bnez a0, skip_context_tracking
	call context_tracking_user_exit
skip_context_tracking:
#endif
	bge s4, zero, 1f
	la ra, ret_from_exception
	move a0, sp
	la a1, handle_arch_irq
	REG_L a1, (a1)
	jr a1
1:
	andi t0, s1, SR_PIE
	beqz t0, 1f
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_on
#endif
	csrs CSR_STATUS, SR_IE
1:
	la ra, ret_from_exception
	li t0, EXC_SYSCALL
	beq s4, t0, handle_syscall
	slli t0, s4, RISCV_LGPTR
	la t1, excp_vect_table
	la t2, excp_vect_table_end
	move a0, sp
	add t0, t1, t0
	bgeu t0, t2, 1f
	REG_L t0, 0(t0)
	jr t0
1:
	tail do_trap_unknown
handle_syscall:
#ifdef CONFIG_RISCV_M_MODE
	csrs CSR_STATUS, SR_IE
#endif
#if defined(CONFIG_TRACE_IRQFLAGS) || defined(CONFIG_CONTEXT_TRACKING)
	REG_L a0, PT_A0(sp)
	REG_L a1, PT_A1(sp)
	REG_L a2, PT_A2(sp)
	REG_L a3, PT_A3(sp)
	REG_L a4, PT_A4(sp)
	REG_L a5, PT_A5(sp)
	REG_L a6, PT_A6(sp)
	REG_L a7, PT_A7(sp)
#endif
	REG_S a0, PT_ORIG_A0(sp)
	addi s2, s2, 0x4
	REG_S s2, PT_EPC(sp)
	REG_L t0, TASK_TI_FLAGS(tp)
	andi t0, t0, _TIF_SYSCALL_WORK
	bnez t0, handle_syscall_trace_enter
check_syscall_nr:
	li t0, __NR_syscalls
	la s0, sys_ni_syscall
	bgeu a7, t0, 1f
	la s0, sys_call_table
	slli t0, a7, RISCV_LGPTR
	add s0, s0, t0
	REG_L s0, 0(s0)
1:
	jalr s0
ret_from_syscall:
	REG_S a0, PT_A0(sp)
ret_from_syscall_rejected:
	REG_L t0, TASK_TI_FLAGS(tp)
	andi t0, t0, _TIF_SYSCALL_WORK
	bnez t0, handle_syscall_trace_exit
ret_from_exception:
	REG_L s0, PT_STATUS(sp)
	csrc CSR_STATUS, SR_IE
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_off
#endif
#ifdef CONFIG_RISCV_M_MODE
	li t0, SR_MPP
	and s0, s0, t0
#else
	andi s0, s0, SR_SPP
#endif
	bnez s0, resume_kernel
resume_userspace:
	REG_L s0, TASK_TI_FLAGS(tp)
	andi s1, s0, _TIF_WORK_MASK
	bnez s1, work_pending
#ifdef CONFIG_CONTEXT_TRACKING
	call context_tracking_user_enter
#endif
	addi s0, sp, PT_SIZE_ON_STACK
	REG_S s0, TASK_TI_KERNEL_SP(tp)
	csrw CSR_SCRATCH, tp
restore_all:
#ifdef CONFIG_TRACE_IRQFLAGS
	REG_L s1, PT_STATUS(sp)
	andi t0, s1, SR_PIE
	beqz t0, 1f
	call __trace_hardirqs_on
	j 2f
1:
	call __trace_hardirqs_off
2:
#endif
	REG_L a0, PT_STATUS(sp)
```

### Cross-File Context

[CALLER_ADDR1 — macro — include/linux/ftrace.h:807]
CALLER_ADDR1 → ((unsigned long)ftrace_return_address(1))  (include/linux/ftrace.h:807)

[__trace_hardirqs_off — helper — arch/riscv/kernel/trace_irq.c:17-21]
```assembly
void __trace_hardirqs_off(void)
{
	trace_hardirqs_off();
}
NOKPROBE_SYMBOL(__trace_hardirqs_off);
```

[__trace_hardirqs_on — helper — arch/riscv/kernel/trace_irq.c:11-15]
```assembly
void __trace_hardirqs_on(void)
{
	trace_hardirqs_on();
}
NOKPROBE_SYMBOL(__trace_hardirqs_on);
```

[NOKPROBE_SYMBOL — macro — include/linux/kprobes.h]
NOKPROBE_SYMBOL → #define NOKPROBE_SYMBOL(sym) __NOKPROBE_LABEL(sym)  (include/linux/kprobes.h)

[ftrace_return_address — function — include/linux/ftrace.h:800]
```assembly
# define ftrace_return_address(n) __builtin_return_address(n)
```

Control flow: handle_exception begins by swapping tp with CSR_SCRATCH to retrieve the task pointer, then sets up the kernel stack and saves all architectural registers. After saving the user SP to the stack and clearing CSR_SCRATCH, it optionally calls __trace_hardirqs_off when CONFIG_TRACE_IRQFLAGS is enabled. Control then branches based on CSR_CAUSE: synchronous exceptions (cause < 0) go directly to ret_from_exception, while interrupts (cause >= 0) check if interrupts were previously enabled and may call __trace_hardirqs_on before enabling interrupts. Exception vectors are looked up in excp_vect_table with bounds checking against excp_vect_table_end. Syscalls have special handling that includes bounds validation of the syscall number against __NR_syscalls before dispatching via sys_call_table. ret_from_exception clears interrupts, optionally calls __trace_hardirqs_off, and determines whether to resume kernel or user mode, with context tracking and TIF flag checks on the return paths.

Data flow: The task pointer (tp) is restored from CSR_SCRATCH at entry and used to access TASK_TI_KERNEL_SP and TASK_TI_USER_SP for stack management. All integer registers and key CSRs (STATUS, EPC, TVAL, CAUSE, SCRATCH) are saved to the stack frame. The syscall number in a7 is bounded against __NR_syscalls before indexing into sys_call_table with slli a7, RISCV_LGPTR for proper addressing. Task flags are checked via TASK_TI_FLAGS(tp) masked with _TIF_SYSCALL_WORK and _TIF_WORK_MASK to determine tracing and work pending behavior. CSR_SCRATCH is explicitly zeroed after tp is extracted and restored to tp before returning to userspace.

Verdict: Not Vulnerable

Reviewed region:
[arch/riscv/kernel/entry.S]
```assembly
.globl handle_exception
handle_exception:
	csrrw tp, CSR_SCRATCH, tp
	bnez tp, _save_context
_restore_kernel_tpsp:
	csrr tp, CSR_SCRATCH
	REG_S sp, TASK_TI_KERNEL_SP(tp)
_save_context:
	REG_S sp, TASK_TI_USER_SP(tp)
	REG_L sp, TASK_TI_KERNEL_SP(tp)
	addi sp, sp, -(PT_SIZE_ON_STACK)
	REG_S x1,  PT_RA(sp)
	REG_S x3,  PT_GP(sp)
	REG_S x5,  PT_T0(sp)
	REG_S x6,  PT_T1(sp)
	REG_S x7,  PT_T2(sp)
	REG_S x8,  PT_S0(sp)
	REG_S x9,  PT_S1(sp)
	REG_S x10, PT_A0(sp)
	REG_S x11, PT_A1(sp)
	REG_S x12, PT_A2(sp)
	REG_S x13, PT_A3(sp)
	REG_S x14, PT_A4(sp)
	REG_S x15, PT_A5(sp)
	REG_S x16, PT_A6(sp)
	REG_S x17, PT_A7(sp)
	REG_S x18, PT_S2(sp)
	REG_S x19, PT_S3(sp)
	REG_S x20, PT_S4(sp)
	REG_S x21, PT_S5(sp)
	REG_S x22, PT_S6(sp)
	REG_S x23, PT_S7(sp)
	REG_S x24, PT_S8(sp)
	REG_S x25, PT_S9(sp)
	REG_S x26, PT_S10(sp)
	REG_S x27, PT_S11(sp)
	REG_S x28, PT_T3(sp)
	REG_S x29, PT_T4(sp)
	REG_S x30, PT_T5(sp)
	REG_S x31, PT_T6(sp)
	li t0, SR_SUM | SR_FS
	REG_L s0, TASK_TI_USER_SP(tp)
	csrrc s1, CSR_STATUS, t0
	csrr s2, CSR_EPC
	csrr s3, CSR_TVAL
	csrr s4, CSR_CAUSE
	csrr s5, CSR_SCRATCH
	REG_S s0, PT_SP(sp)
	REG_S s1, PT_STATUS(sp)
	REG_S s2, PT_EPC(sp)
	REG_S s3, PT_BADADDR(sp)
	REG_S s4, PT_CAUSE(sp)
	REG_S s5, PT_TP(sp)
	csrw CSR_SCRATCH, x0
.option push
.option norelax
	la gp, __global_pointer$
.option pop
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_off
#endif
#ifdef CONFIG_CONTEXT_TRACKING
	li   a0, SR_PP
	and a0, s1, a0
	bnez a0, skip_context_tracking
	call context_tracking_user_exit
skip_context_tracking:
#endif
	bge s4, zero, 1f
	la ra, ret_from_exception
	move a0, sp
	la a1, handle_arch_irq
	REG_L a1, (a1)
	jr a1
1:
	andi t0, s1, SR_PIE
	beqz t0, 1f
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_on
#endif
	csrs CSR_STATUS, SR_IE
1:
	la ra, ret_from_exception
	li t0, EXC_SYSCALL
	beq s4, t0, handle_syscall
	slli t0, s4, RISCV_LGPTR
	la t1, excp_vect_table
	la t2, excp_vect_table_end
	move a0, sp
	add t0, t1, t0
	bgeu t0, t2, 1f
	REG_L t0, 0(t0)
	jr t0
1:
	tail do_trap_unknown
handle_syscall:
#ifdef CONFIG_RISCV_M_MODE
	csrs CSR_STATUS, SR_IE
#endif
#if defined(CONFIG_TRACE_IRQFLAGS) || defined(CONFIG_CONTEXT_TRACKING)
	REG_L a0, PT_A0(sp)
	REG_L a1, PT_A1(sp)
	REG_L a2, PT_A2(sp)
	REG_L a3, PT_A3(sp)
	REG_L a4, PT_A4(sp)
	REG_L a5, PT_A5(sp)
	REG_L a6, PT_A6(sp)
	REG_L a7, PT_A7(sp)
#endif
	REG_S a0, PT_ORIG_A0(sp)
	addi s2, s2, 0x4
	REG_S s2, PT_EPC(sp)
	REG_L t0, TASK_TI_FLAGS(tp)
	andi t0, t0, _TIF_SYSCALL_WORK
	bnez t0, handle_syscall_trace_enter
check_syscall_nr:
	li t0, __NR_syscalls
	la s0, sys_ni_syscall
	bgeu a7, t0, 1f
	la s0, sys_call_table
	slli t0, a7, RISCV_LGPTR
	add s0, s0, t0
	REG_L s0, 0(s0)
1:
	jalr s0
ret_from_syscall:
	REG_S a0, PT_A0(sp)
ret_from_syscall_rejected:
	REG_L t0, TASK_TI_FLAGS(tp)
	andi t0, t0, _TIF_SYSCALL_WORK
	bnez t0, handle_syscall_trace_exit
ret_from_exception:
	REG_L s0, PT_STATUS(sp)
	csrc CSR_STATUS, SR_IE
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_off
#endif
#ifdef CONFIG_RISCV_M_MODE
	li t0, SR_MPP
	and s0, s0, t0
#else
	andi s0, s0, SR_SPP
#endif
	bnez s0, resume_kernel
resume_userspace:
	REG_L s0, TASK_TI_FLAGS(tp)
	andi s1, s0, _TIF_WORK_MASK
	bnez s1, work_pending
#ifdef CONFIG_CONTEXT_TRACKING
	call context_tracking_user_enter
#endif
	addi s0, sp, PT_SIZE_ON_STACK
	REG_S s0, TASK_TI_KERNEL_SP(tp)
	csrw CSR_SCRATCH, tp
restore_all:
#ifdef CONFIG_TRACE_IRQFLAGS
	REG_L s1, PT_STATUS(sp)
	andi t0, s1, SR_PIE
	beqz t0, 1f
	call __trace_hardirqs_on
	j 2f
1:
	call __trace_hardirqs_off
2:
#endif
	REG_L a0, PT_STATUS(sp)
```

Why it is not vulnerable: The task pointer (tp) register, which serves as the thread pointer, is properly initialized at entry via csrrw tp, CSR_SCRATCH, tp and csrr tp, CSR_SCRATCH, and remains unchanged throughout the execution paths that call __trace_hardirqs_off and __trace_hardirqs_on. The csrw CSR_SCRATCH, x0 instruction after the register save block only clears the scratch CSR and does not affect tp. All trace function calls occur on a valid kernel stack with tp pointing to valid task memory. Syscall dispatch properly bounds-checks a7 against __NR_syscalls before table lookup. Exception vector table lookups are bounded by excp_vect_table_end.

Security invariants:
- tp (task/thread pointer) must remain valid when trace functions are called; enforced by initializing tp at entry (csrrw tp, CSR_SCRATCH, tp) and not modifying it before __trace_hardirqs_off or __trace_hardirqs_on calls
- Syscall number must be bounded before table indexing; enforced by bgeu a7, t0, 1f where t0 is __NR_syscalls, defaulting to sys_ni_syscall on out-of-range
- Exception vector table lookup must be bounds-checked; enforced by computing the offset with slli t0, s4, RISCV_LGPTR and checking bgeu t0, t2 where t2 is excp_vect_table_end, falling back to do_trap_unknown on out-of-range
- Kernel stack must be valid when registers are saved and functions are called; enforced by REG_L sp, TASK_TI_KERNEL_SP(tp) followed by addi sp, sp, -(PT_SIZE_ON_STACK) to allocate the stack frame
- CSR_SCRATCH must be restored before returning to userspace; enforced by csrw CSR_SCRATCH, tp at resume_userspace before restore_all

Invariant verification:
- tp register validity before __trace_hardirqs_off call: holds=true. Evidence: tp is loaded via csrrw tp, CSR_SCRATCH, tp and csrr tp, CSR_SCRATCH at the start of handle_exception, and the subsequent csrw CSR_SCRATCH, x0 only clears the CSR, not tp; no instruction between tp initialization and the call __trace_hardirqs_off modifies tp
- tp register validity before __trace_hardirqs_on call: holds=true. Evidence: The same tp value initialized at entry flows through all paths to __trace_hardirqs_on; no intervening code modifies tp between initialization and the call site
- Syscall number bounds validation before table dispatch: holds=true. Evidence: li t0, __NR_syscalls followed by bgeu a7, t0, 1f ensures out-of-range syscall numbers fall through to sys_ni_syscall rather than performing an unbounded table lookup
- Exception vector table bounds checking: holds=true. Evidence: la t1, excp_vect_table; la t2, excp_vect_table_end; bgeu t0, t2, 1f ensures the computed table offset does not exceed the table bounds, with fallback to tail do_trap_unknown
- Kernel stack frame properly allocated before register saves and function calls: holds=true. Evidence: REG_L sp, TASK_TI_KERNEL_SP(tp) loads kernel stack pointer, then addi sp, sp, -(PT_SIZE_ON_STACK) reserves stack space before all register saves and trace function invocations

Residual risk notes: The __trace_hardirqs_off and __trace_hardirqs_on helper functions invoke trace_hardirqs_off() and trace_hardirqs_on() respectively, which may perform stack walking using CALLER_ADDR1 that ultimately relies on ftrace_return_address(1) / __builtin_return_address(1). While this mechanism uses the return address on the stack rather than a frame pointer walk, the correctness of stack frame layout and return address preservation depends on proper calling convention adherence by all intermediate call chain functions. Additionally, the context_tracking_user_exit and context_tracking_user_enter calls similarly may perform operations that depend on valid execution context state.

An analysis of the control and data flow reveals a vulnerability regarding stack unwinding and register conventions in the kernel entry path.

### Vulnerability Explanation
In `handle_exception`, the code saves the user registers and sets up the kernel stack. During this process, it executes:
```assembly
	REG_L s0, TASK_TI_USER_SP(tp)
        ...
#ifdef CONFIG_TRACE_IRQFLAGS
	call __trace_hardirqs_off
#endif
```
In the RISC-V ABI, `s0` serves as the frame pointer (`fp`). Here, `s0` is loaded with the user-space stack pointer (an attacker-controlled value) just before calling `__trace_hardirqs_off`.

`__trace_hardirqs_off` is a standard C function that calls `trace_hardirqs_off()`. Under the hood, `trace_hardirqs_off()` uses `CALLER_ADDR1` (which resolves to `__builtin_return_address(1)`) to record who disabled the interrupts. To resolve `__builtin_return_address(1)`, the compiler generates code that walks the frame pointer linked list.

Since `handle_exception` does not set up a standard stack frame and leaves the user-controlled stack pointer in `s0` (`fp`), the unwinder inside `trace_hardirqs_off()` will dereference the user stack pointer in an attempt to read the caller's return address and previous frame pointer.

### Impact and Exploitability
Because this occurs in kernel mode, dereferencing an unvalidated user-space pointer can trigger a supervisor page fault (especially if Supervisor Mode Access Prevention / SUM is active, or if the address is unmapped).
An unprivileged local attacker can trivially trigger this by setting `sp` to an invalid or unmapped address and issuing a system call or causing an exception. This results in an immediate kernel panic, leading to a local Denial of Service (DoS).

### Verdict
**Vulnerable:** Local Denial of Service (DoS) / Kernel Panic via arbitrary frame pointer dereference during `__builtin_return_address(1)` resolution in the interrupt tracing path.

```json
{
  "vulnerable": true,
  "cwe": "CWE-119",
  "severity": "Medium",
  "reviewed_region": "arch/riscv/kernel/entry.S",
  "vulnerable_region": "handle_exception",
  "root_cause": "The frame pointer register `s0` (fp) is loaded with the user-space stack pointer and left uncleared before calling C functions (`__trace_hardirqs_off`). When the C code attempts to unwind the stack using `__builtin_return_address()`, it dereferences the user-controlled pointer.",
  "exploitability": "Trivial for a local user. An attacker simply sets the stack pointer to an invalid address and triggers an exception (e.g., via syscall).",
  "attack_preconditions": "The kernel must be compiled with `CONFIG_TRACE_IRQFLAGS` enabled.",
  "impact": "Local Denial of Service (DoS) due to kernel panic upon dereferencing an invalid or unmapped user-space pointer during the stack walk.",
  "non_vulnerable_reason": "",
  "security_invariants": "",
  "invariant_verification": "",
  "residual_risk_notes": "",
  "call_path": "handle_exception (arch/riscv/kernel/entry.S) \u2192 trace_hardirqs_off (kernel/trace/trace_preemptirq.c) \u2192 trace_hardirqs_on (kernel/trace/trace_preemptirq.c) \u2192 __trace_hardirqs_off (arch/riscv/kernel/trace_irq.c) \u2192 __trace_hardirqs_on (arch/riscv/kernel/trace_irq.c)"
}
```


Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: github.com/k3s-io/k3s
Language: Go
File: pkg/daemons/agent/agent.go
Function: defaultKubeletConfig

Call path: kubeletArgsAndConfig (pkg/daemons/agent/agent_linux.go) → defaultKubeletConfig (pkg/daemons/agent/agent.go) → writeKubeletConfig (pkg/daemons/agent/agent.go) → yaml.Marshal (builtin)

### Primary Function

```go
func defaultKubeletConfig(cfg *daemonconfig.Agent) (*kubeletconfig.KubeletConfiguration, error) {
	bindAddress := "127.0.0.1"
	isIPv6 := utilsnet.IsIPv6(net.ParseIP([]string{cfg.NodeIP}[0]))
	if isIPv6 {
		bindAddress = "::1"
	}

	defaultConfig := &kubeletconfig.KubeletConfiguration{
		TypeMeta: metav1.TypeMeta{
			APIVersion: "kubelet.config.k8s.io/v1beta1",
			Kind:       "KubeletConfiguration",
		},
		CPUManagerReconcilePeriod:        metav1.Duration{Duration: time.Second * 10},
		CgroupDriver:                     "cgroupfs",
		ClusterDomain:                    cfg.ClusterDomain,
		EvictionPressureTransitionPeriod: metav1.Duration{Duration: time.Minute * 5},
		FailSwapOn:                       utilsptr.To(false),
		FileCheckFrequency:               metav1.Duration{Duration: time.Second * 20},
		HTTPCheckFrequency:               metav1.Duration{Duration: time.Second * 20},
		HealthzBindAddress:               bindAddress,
		ImageMinimumGCAge:                metav1.Duration{Duration: time.Minute * 2},
		NodeStatusReportFrequency:        metav1.Duration{Duration: time.Minute * 5},
		NodeStatusUpdateFrequency:        metav1.Duration{Duration: time.Second * 10},
		ProtectKernelDefaults:            cfg.ProtectKernelDefaults,
		ReadOnlyPort:                     0,
		RuntimeRequestTimeout:            metav1.Duration{Duration: time.Minute * 2},
		StreamingConnectionIdleTimeout:   metav1.Duration{Duration: time.Hour * 4},
		SyncFrequency:                    metav1.Duration{Duration: time.Minute},
		VolumeStatsAggPeriod:             metav1.Duration{Duration: time.Minute},
		EvictionHard: map[string]string{
			"imagefs.available": "5%",
			"nodefs.available":  "5%",
		},
		EvictionMinimumReclaim: map[string]string{
			"imagefs.available": "10%",
			"nodefs.available":  "10%",
		},
		Authentication: kubeletconfig.KubeletAuthentication{
			Anonymous: kubeletconfig.KubeletAnonymousAuthentication{
				Enabled: utilsptr.To(false),
			},
			Webhook: kubeletconfig.KubeletWebhookAuthentication{
				Enabled:  utilsptr.To(true),
				CacheTTL: metav1.Duration{Duration: time.Minute * 2},
			},
		},
		Authorization: kubeletconfig.KubeletAuthorization{
			Mode: kubeletconfig.KubeletAuthorizationModeWebhook,
			Webhook: kubeletconfig.KubeletWebhookAuthorization{
				CacheAuthorizedTTL:   metav1.Duration{Duration: time.Minute * 5},
				CacheUnauthorizedTTL: metav1.Duration{Duration: time.Second * 30},
			},
		},
		Logging: logsv1.LoggingConfiguration{
			Format:    "text",
			Verbosity: logsv1.VerbosityLevel(cfg.VLevel),
			FlushFrequency: logsv1.TimeOrMetaDuration{
				Duration:          metav1.Duration{Duration: time.Second * 5},
				SerializeAsString: true,
			},
		},
	}

	if cfg.ListenAddress != "" {
		defaultConfig.Address = cfg.ListenAddress
	}

	if cfg.ClientCA != "" {
		defaultConfig.Authentication.X509.ClientCAFile = cfg.ClientCA
	}

	if cfg.ServingKubeletCert != "" && cfg.ServingKubeletKey != "" {
		defaultConfig.TLSCertFile = cfg.ServingKubeletCert
		defaultConfig.TLSPrivateKeyFile = cfg.ServingKubeletKey
	}

	for _, addr := range cfg.ClusterDNSs {
		defaultConfig.ClusterDNS = append(defaultConfig.ClusterDNS, addr.String())
	}

	if cfg.ResolvConf != "" {
		defaultConfig.ResolverConfig = utilsptr.To(cfg.ResolvConf)
	}

	if cfg.PodManifests != "" && defaultConfig.StaticPodPath == "" {
		defaultConfig.StaticPodPath = cfg.PodManifests
	}
	if err := os.MkdirAll(defaultConfig.StaticPodPath, 0750); err != nil {
		return nil, pkgerrors.WithMessagef(err, "failed to create static pod manifest dir %s", defaultConfig.StaticPodPath)
	}

	if t, _, err := taints.ParseTaints(cfg.NodeTaints); err != nil {
		return nil, pkgerrors.WithMessage(err, "failed to parse node taints")
	} else {
		defaultConfig.RegisterWithTaints = t
	}

	logsv1.VModuleConfigurationPflag(&defaultConfig.Logging.VModule).Set(cfg.VModule)

	return defaultConfig, nil
}
```

### Cross-File Context

[writeKubeletConfig — helper — pkg/daemons/agent/agent.go:156-162]
```go
func writeKubeletConfig(path string, config *kubeletconfig.KubeletConfiguration) error {
	b, err := yaml.Marshal(config)
	if err != nil {
		return err
	}
	return os.WriteFile(filepath.Join(path, "00-"+version.Program+"-defaults.conf"), b, 0600)
}
```

[KubeletConfiguration — type — k8s.io/kubelet/config/v1beta1]
kubeletconfig.KubeletConfiguration (from k8s.io/kubelet/config/v1beta1)

Control flow: kubeletArgsAndConfig calls defaultKubeletConfig to create a KubeletConfiguration struct, then writeKubeletConfig marshals it to YAML and writes to file. The YAML marshaller omits zero values, so ReadOnlyPort=0 is not written to config.

Data flow: ReadOnlyPort=0 is set in struct literal, passed to writeKubeletConfig, marshalled to YAML (where zero is omitted), and written to config file. The kubelet never receives the explicit 0 value.

Verdict: Vulnerable
Vulnerable region:
[pkg/daemons/agent/agent.go]
```go
defaultConfig := &kubeletconfig.KubeletConfiguration{
		TypeMeta: metav1.TypeMeta{
			APIVersion: "kubelet.config.k8s.io/v1beta1",
			Kind:       "KubeletConfiguration",
		},
		CPUManagerReconcilePeriod:        metav1.Duration{Duration: time.Second * 10},
		CgroupDriver:                     "cgroupfs",
		ClusterDomain:                    cfg.ClusterDomain,
		EvictionPressureTransitionPeriod: metav1.Duration{Duration: time.Minute * 5},
		FailSwapOn:                       utilsptr.To(false),
		FileCheckFrequency:               metav1.Duration{Duration: time.Second * 20},
		HTTPCheckFrequency:               metav1.Duration{Duration: time.Second * 20},
		HealthzBindAddress:               bindAddress,
		ImageMinimumGCAge:                metav1.Duration{Duration: time.Minute * 2},
		NodeStatusReportFrequency:        metav1.Duration{Duration: time.Minute * 5},
		NodeStatusUpdateFrequency:        metav1.Duration{Duration: time.Second * 10},
		ProtectKernelDefaults:            cfg.ProtectKernelDefaults,
		ReadOnlyPort:                     0,
		RuntimeRequestTimeout:            metav1.Duration{Duration: time.Minute * 2},
		StreamingConnectionIdleTimeout:   metav1.Duration{Duration: time.Hour * 4},
		SyncFrequency:                    metav1.Duration{Duration: time.Minute},
		VolumeStatsAggPeriod:             metav1.Duration{Duration: time.Minute},
		EvictionHard: map[string]string{
			"imagefs.available": "5%",
			"nodefs.available":  "5%",
		},
		EvictionMinimumReclaim: map[string]string{
			"imagefs.available": "10%",
			"nodefs.available":  "10%",
		},
		Authentication: kubeletconfig.KubeletAuthentication{
			Anonymous: kubeletconfig.KubeletAnonymousAuthentication{
				Enabled: utilsptr.To(false),
			},
			Webhook: kubeletconfig.KubeletWebhookAuthentication{
				Enabled:  utilsptr.To(true),
				CacheTTL: metav1.Duration{Duration: time.Minute * 2},
			},
		},
		Authorization: kubeletconfig.KubeletAuthorization{
			Mode: kubeletconfig.KubeletAuthorizationModeWebhook,
			Webhook: kubeletconfig.KubeletWebhookAuthorization{
				CacheAuthorizedTTL:   metav1.Duration{Duration: time.Minute * 5},
				CacheUnauthorizedTTL: metav1.Duration{Duration: time.Second * 30},
			},
		},
		Logging: logsv1.LoggingConfiguration{
			Format:    "text",
			Verbosity: logsv1.VerbosityLevel(cfg.VLevel),
			FlushFrequency: logsv1.TimeOrMetaDuration{
				Duration:          metav1.Duration{Duration: time.Second * 5},
				SerializeAsString: true,
			},
		},
```
CWE: CWE-1188
Severity: MEDIUM

Root cause: In defaultKubeletConfig(), the ReadOnlyPort field is set to 0 in the KubeletConfiguration struct. When writeKubeletConfig() marshals this struct to YAML using yaml.Marshal(), zero values are omitted from the output. This means the kubelet would not receive any read-only port configuration from the config file, causing it to use an unintended default value (likely non-zero), creating an unintended unauthenticated endpoint.

Exploitability: An attacker with network access to the kubelet read-only port (default 10255) can retrieve sensitive node and pod information without authentication. The read-only port may be unintentionally exposed when the YAML configuration omits the zero-value ReadOnlyPort field.

Attack preconditions:
- Network access to kubelet read-only port (10255) on a k3s node
- Kubelet not explicitly configured to disable read-only port via both CLI flag and config file

Likely impact: An attacker with network access to the kubelet read-only port (default 10255) can retrieve sensitive node and pod information without authentication. The read-only port may be unintentionally exposed when the YAML configuration omits the zero-value ReadOnlyPort field.

kubeletArgsAndConfig defaultKubeletConfig writeKubeletConfig yaml.Marshal

Based on a security analysis of the provided code, there are no exploitable vulnerabilities.

### Analysis
The function `defaultKubeletConfig` is responsible for building a `KubeletConfiguration` object from the provided `cfg *daemonconfig.Agent` struct. The control and data flow show standard initialization logic:
1. **Network Configuration:** It attempts to detect if the node IP is IPv6 to set the `HealthzBindAddress`.
   *(Note: The construct `[]string{cfg.NodeIP}[0]` does not split a comma-separated dual-stack IP string as the developer likely intended—they likely meant `strings.Split(cfg.NodeIP, ",")[0]`. This will cause `net.ParseIP` to return `nil` on dual-stack IP configurations, but this is an operational bug, not a security vulnerability).*
2. **Kubelet Configuration Struct:** The struct is populated with standard Kubernetes configuration defaults. Features such as anonymous authentication are appropriately disabled, and webhook authentication/authorization are enabled.
3. **Directory Creation:** The code uses `os.MkdirAll` to create the static pod manifest path with secure `0750` permissions.
4. **Serialization and Persistence:** The `writeKubeletConfig` function uses Go's `yaml.Marshal` which safely serializes the structural data, preventing configuration injection, and writes the file with securely restricted permissions (`0600`).

### Conclusion
The code securely configures the Kubelet environment, disables anonymous access by default, and safely serializes and writes the configuration file using secure file permissions. No security flaws (such as injection, path traversal, or privilege escalation) are present.

```json
{
  "vulnerable": false,
  "cwe": null,
  "severity": null,
  "reviewed_region": "func defaultKubeletConfig(cfg *daemonconfig.Agent) (*kubeletconfig.KubeletConfiguration, error) {\n\tbindAddress := \"127.0.0.1\"\n\tisIPv6 := utilsnet.IsIPv6(net.ParseIP([]string{cfg.NodeIP}[0]))\n\tif isIPv6 {\n\t\tbindAddress = \"::1\"\n\t}\n\n\tdefaultConfig := &kubeletconfig.KubeletConfiguration{\n        ...",
  "vulnerable_region": null,
  "root_cause": null,
  "exploitability": null,
  "attack_preconditions": null,
  "impact": null,
  "non_vulnerable_reason": "The function initializes a structured KubeletConfiguration object and populates it with variables from a configuration object. It disables anonymous authentication by default, ensures secure file and directory permissions (0600 and 0750 respectively), and avoids injection via safe YAML marshaling of the structure.",
  "security_invariants": [
    "YAML marshaling uses typed Go structures preventing injection",
    "Sensitive files are written with 0600 permissions",
    "Directories are created with 0750 permissions",
    "Kubelet anonymous authentication is explicitly disabled"
  ],
  "invariant_verification": "The writeKubeletConfig helper explicitly uses `0600` for `os.WriteFile`, and `defaultKubeletConfig` uses `0750` for `os.MkdirAll`. `yaml.Marshal` handles structural encoding. Kubelet Anonymous Authentication is set to `false`.",
  "residual_risk_notes": "The construct `[]string{cfg.NodeIP}[0]` fails to properly parse dual-stack IP strings, but this causes an operational error rather than a security compromise.",
  "call_path": "kubeletArgsAndConfig (pkg/daemons/agent/agent_linux.go) \u2192 defaultKubeletConfig (pkg/daemons/agent/agent.go) \u2192 writeKubeletConfig (pkg/daemons/agent/agent.go) \u2192 yaml.Marshal (builtin)"
}
```

func defaultKubeletConfig(cfg *daemonconfig.Agent) (*kubeletconfig.KubeletConfiguration, error) {
	bindAddress := "127.0.0.1"
	isIPv6 := utilsnet.IsIPv6(net.ParseIP([]string{cfg.NodeIP}[0]))
	if isIPv6 {
		bindAddress = "::1"
	}

	defaultConfig := &kubeletconfig.KubeletConfiguration{
        ...


Audit the following candidate code to determine whether it presents a real vulnerability.
This sample may come from a noisy retrieval or transformation stage, so reason from the supplied code and context rather than assuming the label.

Repository: github.com/cakephp/cakephp
Language: PHP
File: cake/libs/controller/components/security.php
Function: _validatePost

Call path: SecurityComponent::startup (cake/libs/controller/components/security.php) → SecurityComponent::_validatePost (cake/libs/controller/components/security.php) → unserialize (PHP builtin)

### Primary Function

```php
function _validatePost(&$controller) {
	if (empty($controller->data)) {
		return true;
	}
	$data = $controller->data;

	if (!isset($data['_Token']) || !isset($data['_Token']['fields']) || !isset($data['_Token']['key'])) {
		return false;
	}
	$token = $data['_Token']['key'];

	if ($this->Session->check('_Token')) {
		$tokenData = unserialize($this->Session->read('_Token'));

		if ($tokenData['expires'] < time() || $tokenData['key'] !== $token) {
			return false;
		}
	}

	$locked = null;
	$check = $controller->data;
	$token = urldecode($check['_Token']['fields']);

	if (strpos($token, ':')) {
		list($token, $locked) = explode(':', $token, 2);
	}
	unset($check['_Token']);

	$lockedFields = array();
	$fields = Set::flatten($check);
	$fieldList = array_keys($fields);
	$locked = unserialize(str_rot13($locked));
	$multi = array();

	foreach ($fieldList as $i => $key) {
		if (preg_match('/\.\d+$/', $key)) {
			$multi[$i] = preg_replace('/\.\d+$/', '', $key);
			unset($fieldList[$i]);
		}
	}
	if (!empty($multi)) {
		$fieldList += array_unique($multi);
	}

	foreach ($fieldList as $i => $key) {
		$isDisabled = false;
		$isLocked = (is_array($locked) && in_array($key, $locked));

		if (!empty($this->disabledFields)) {
			foreach ((array)$this->disabledFields as $disabled) {
				$disabled = explode('.', $disabled);
				$field = array_values(array_intersect(explode('.', $key), $disabled));
				$isDisabled = ($field === $disabled);
				if ($isDisabled) {
					break;
				}
			}
		}

		if ($isDisabled || $isLocked) {
			unset($fieldList[$i]);
			if ($isLocked) {
				$lockedFields[$key] = $fields[$key];
			}
		}
	}
	sort($fieldList, SORT_STRING);
	ksort($lockedFields, SORT_STRING);

	$fieldList += $lockedFields;
	$check = Security::hash(serialize($fieldList) . Configure::read('Security.salt'));
	return ($token === $check);
}
```

### Cross-File Context

[SecurityComponent — class — cake/libs/controller/components/security.php:33]
class SecurityComponent extends Object

[unserialize — sink — PHP builtin]
unserialize(string $data): mixed

[str_rot13 — builtin — PHP builtin]
str_rot13(string $data): string

[Set::flatten — method — cake/libs/set.php:1036]
function flatten($data, $separator = '.')

[Security::hash — method — cake/libs/security.php]
function hash($data, $type = null)

Control flow: The _validatePost method validates form tokens by extracting the 'fields' parameter from POST data, URL-decoding it, splitting on ':' to get the 'locked' value, then unserializing the rot13-decoded locked value to check which fields are locked.

Data flow: User-controlled POST data flows through: $controller->data['_Token']['fields'] -> urldecode() -> explode() -> str_rot13() -> unserialize(). The tainted $locked variable reaches the unsafe unserialize() call at line 624 without validation.

Verdict: Vulnerable
Vulnerable region:
[cake/libs/controller/components/security.php]
```php
$locked = unserialize(str_rot13($locked));
```
CWE: CWE-20
Severity: HIGH

Root cause: The _validatePost method accepts serialized data from the _Token['fields'] POST parameter. The locked field is extracted, rot13 decoded, and then directly passed to unserialize() without any validation. This allows an attacker to send crafted serialized objects that could manipulate file map caches or execute arbitrary code when deserialized.

Exploitability: An unauthenticated attacker can craft a POST request with a malicious serialized object in the _Token[fields] parameter. The object is rot13 encoded before sending, then decoded and unserialized without validation on the server side.

Attack preconditions:
- Attacker can send HTTP POST requests to a CakePHP endpoint using SecurityComponent
- Target application has form token validation enabled via SecurityComponent
- Attacker can craft a rot13-encoded serialized PHP object containing malicious payload

Likely impact: An unauthenticated attacker can craft a POST request with a malicious serialized object in the _Token[fields] parameter. The object is rot13 encoded before sending, then decoded and unserialized without validation on the server side.

An analysis of the `_validatePost` function reveals a critical security vulnerability due to the unsafe deserialization of user-controlled data.

### Vulnerable Behavior and Root Cause
In the function, the POST payload (`$controller->data`) is inspected. Specifically, the `_Token.fields` parameter is extracted, URL-decoded, and split into two variables (`$token` and `$locked`) based on a colon `:` delimiter:

```php
$check = $controller->data;
$token = urldecode($check['_Token']['fields']);

if (strpos($token, ':')) {
	list($token, $locked) = explode(':', $token, 2);
}
// ...
$locked = unserialize(str_rot13($locked));
```

The `$locked` variable, which is entirely controlled by the user, is passed through `str_rot13()` and then directly supplied to PHP's native `unserialize()` function.

While the application eventually attempts to verify the integrity of the data using an HMAC at the very end of the function (`$check = Security::hash(...)`), the `unserialize()` call happens **before** this signature validation. This means the application blindly deserializes whatever the user inputs, rendering the subsequent HMAC check useless for preventing object injection.

### Exploitability and Impact
An attacker can easily exploit this vulnerability. The only precondition is that the application does not reject the request early, which usually just requires a valid CSRF token (`_Token.key`). Any user who can access a form protected by the `SecurityComponent` will receive a valid CSRF token.

The attacker can then submit a manipulated POST request where `_Token[fields]` contains a malicious payload crafted as `dummy_token_hash:ROT13(SERIALIZED_PAYLOAD)`. The `unserialize()` function will instantiate the attacker-supplied PHP objects. Depending on the classes available in the application's runtime (or its dependencies) that implement "magic methods" like `__destruct()`, `__wakeup()`, or `__toString()`, this PHP Object Injection (POI) can be leveraged to achieve **Remote Code Execution (RCE)**, arbitrary file deletion, or other severe impacts.

### Vulnerability Classification
**CWE-502:** Deserialization of Untrusted Data (PHP Object Injection).

```json
{
  "vulnerable": true,
  "cwe": "CWE-502",
  "severity": "Critical",
  "reviewed_region": "cake/libs/controller/components/security.php",
  "vulnerable_region": "if (strpos($token, ':')) {\n\t\tlist($token, $locked) = explode(':', $token, 2);\n\t}\n\tunset($check['_Token']);\n\n\t$lockedFields = array();\n\t$fields = Set::flatten($check);\n\t$fieldList = array_keys($fields);\n\t$locked = unserialize(str_rot13($locked));",
  "root_cause": "The application calls `unserialize()` on the user-controlled `$locked` variable (derived from `$controller->data['_Token']['fields']`) prior to verifying the HMAC signature of the data.",
  "exploitability": "An attacker can submit a POST request with a crafted `data[_Token][fields]` parameter containing a ROT13-encoded serialized PHP object payload separated by a colon. Because deserialization occurs before the signature check, the payload will be executed.",
  "attack_preconditions": "The attacker must be able to submit a POST request to an endpoint protected by the SecurityComponent. They also need a valid CSRF token (`_Token.key`), which is normally issued to any user loading the form.",
  "impact": "Remote Code Execution (RCE) via PHP Object Injection, assuming suitable gadget chains are present in the application's autoloaded classes.",
  "non_vulnerable_reason": "",
  "security_invariants": "",
  "invariant_verification": "",
  "residual_risk_notes": "",
  "call_path": "SecurityComponent::startup (cake/libs/controller/components/security.php) \u2192 SecurityComponent::_validatePost (cake/libs/controller/components/security.php) \u2192 unserialize (PHP builtin)"
}
```