hak5 / nano-tetra-modules

The Official WiFi Pineapple Module Repository for the NANO & TETRA
https://wifipineapple.com/modules
463 stars 143 forks source link

OpenVPNConnect 1.3 Update - RCE Bug Fix and Dependency Handling Bug Fix #76

Closed 3ndG4me closed 5 years ago

3ndG4me commented 5 years ago

An RCE existed due improperly sanitized parameters in the config name and optional flags parameter. While it is true that once you are logged into a wifi pineapple, you should technically already have the root password, this was not intended functionality and is a vulnerability that should not exist.

Additionally, upstream improvements to firmware 2.6.x were made that resolve Issue #12 on https://github.com/InjectionSoftwareDevelopment/OpenVPNConnect, but unfortunately a bug was introduced at some point that caused dependency installation and handling to result in a 500 error response. This showed its face in the latest firmware release and was resolved by properly passing a default "false" parameter to the appropriate API function for handling dependencies.