# Unsafe YAML Deserialization via yaml.load with Full Loader

Language: Python
Severity: Critical
CWE: CWE-502

## Source
5

## Flow
5-6

## Sink
6

## Vulnerable Code
```python
import yaml
from flask import request, jsonify

def apply_iot_device_config():
    device_payload = request.data.decode('utf-8')
    parsed_config = yaml.load(device_payload, Loader=yaml.FullLoader)
    device_id = parsed_config.get('device_id')
    firmware_ver = parsed_config.get('firmware_version')
    telemetry_interval = parsed_config.get('telemetry_interval', 60)
    update_device_registry(device_id, firmware_ver, telemetry_interval)
    return jsonify({'status': 'configuration applied', 'device': device_id})
```

## Explanation

The code uses yaml.load() with FullLoader on untrusted user input from request.data without validation. FullLoader can instantiate arbitrary Python objects, allowing attackers to execute arbitrary code through specially crafted YAML payloads containing malicious object constructors.

## Remediation

The fix replaces yaml.load() with yaml.safe_load(), which only deserializes standard YAML tags (strings, numbers, lists, dicts) and refuses to instantiate arbitrary Python objects. Additionally, a type check ensures the parsed result is a dictionary before accessing keys, preventing unexpected behavior from non-mapping YAML inputs.

## Secure Code
```python
import yaml
from flask import request, jsonify

def apply_iot_device_config():
    device_payload = request.data.decode('utf-8')
    parsed_config = yaml.safe_load(device_payload)
    if not isinstance(parsed_config, dict):
        return jsonify({'status': 'error', 'message': 'Invalid configuration format'}), 400
    device_id = parsed_config.get('device_id')
    firmware_ver = parsed_config.get('firmware_version')
    telemetry_interval = parsed_config.get('telemetry_interval', 60)
    update_device_registry(device_id, firmware_ver, telemetry_interval)
    return jsonify({'status': 'configuration applied', 'device': device_id})
```
