Michael Sauer
2e90b8fd06
M2: WLAN-Passwort muss leer (offenes Netz) oder 8..64 Zeichen lang sein (400 "invalid password" bei 1..7 Zeichen). CMake prüft NINLIGHT_WIFI_PASSWORD ebenso (0 oder 8..63 Zeichen) und der Zeichen-Blacklist-Check bekommt `;` dazu. M4: Ein Authorization-Wert, der länger als NL_TOKEN_LEN-1 (64) ist, wird von nl_http_parse() nicht mehr stillschweigend abgeschnitten, sondern setzt `auth_too_long` in nl_http_request_t; nl_api_handle() behandelt eine solche Anfrage als unauthentifiziert (außer bei offenem Zugriff), damit ein abgeschnittener Wert nie zufällig auf ein kürzeres Token passt. M5: get_int() prüfte die (long)-Cast-Gleichheit vor dem Bereichscheck - bei sehr großen oder NaN-Werten ist der Cast von double nach long undefiniertes Verhalten. Bereichs- und NaN-Check laufen jetzt vor dem Cast. T1: sdk/Platform/Drivers/internalflash/wm_internal_fls.c: tls_fls_init() ignorierte den Rückgabewert von tls_os_sem_create(&s_fls2_lock, 1) - jetzt eine TLS_DBGPRT_ERR-Meldung, falls das fehlschlägt (dokumentiert in sdk/PATCHES.md). Tests: test_api.c (Passwortlängen, auth_too_long, 1e300-Brightness), test_http.c (auth_too_long wird gesetzt). Für M5 zeigt der Host-Test kein sichtbares RED, weil der (double)->(long)-Cast auf dieser Plattform/diesem Compiler keinen Crash auslöst (implementation-defined statt UB-Crash) - der Fix wird trotzdem angewendet, da er auf dem Zielcompiler (arm-none-eabi 4.9) nicht garantiert sicher ist.
104 lines
3.4 KiB
CMake
104 lines
3.4 KiB
CMake
cmake_minimum_required(VERSION 3.16)
|
|
|
|
if(EXISTS ${CMAKE_SOURCE_DIR}/config.local.cmake)
|
|
include(${CMAKE_SOURCE_DIR}/config.local.cmake)
|
|
endif()
|
|
|
|
project(ninlight C ASM)
|
|
|
|
if(NOT CMAKE_BUILD_TYPE)
|
|
set(CMAKE_BUILD_TYPE Release)
|
|
endif()
|
|
|
|
set(NINLIGHT_DEVICES evk e27_rgb e14_rgb gu10_rgb e27_cw e14_cw gu10_cw g95_cw)
|
|
set(NINLIGHT_DEVICE "" CACHE STRING "Target device")
|
|
if(NOT NINLIGHT_DEVICE IN_LIST NINLIGHT_DEVICES)
|
|
message(FATAL_ERROR "Set -DNINLIGHT_DEVICE=<one of: ${NINLIGHT_DEVICES}>")
|
|
endif()
|
|
|
|
if(NOT DEFINED NINLIGHT_TOKEN)
|
|
set(NINLIGHT_TOKEN "aabbccddeeff0011223344")
|
|
endif()
|
|
|
|
if(NOT DEFINED NINLIGHT_WIFI_SSID)
|
|
set(NINLIGHT_WIFI_SSID "")
|
|
endif()
|
|
if(NOT DEFINED NINLIGHT_WIFI_PASSWORD)
|
|
set(NINLIGHT_WIFI_PASSWORD "")
|
|
endif()
|
|
if(NOT DEFINED NINLIGHT_AP_PASSWORD)
|
|
set(NINLIGHT_AP_PASSWORD "ninlight")
|
|
endif()
|
|
string(LENGTH "${NINLIGHT_AP_PASSWORD}" _nl_ap_len)
|
|
if(_nl_ap_len LESS 8)
|
|
message(FATAL_ERROR "NINLIGHT_AP_PASSWORD needs at least 8 characters (WPA2)")
|
|
endif()
|
|
# M2: WPA2 station passwords must be empty (open network) or 8..63 chars.
|
|
string(LENGTH "${NINLIGHT_WIFI_PASSWORD}" _nl_wifi_pass_len)
|
|
if(NOT _nl_wifi_pass_len EQUAL 0 AND (_nl_wifi_pass_len LESS 8 OR _nl_wifi_pass_len GREATER 63))
|
|
message(FATAL_ERROR "NINLIGHT_WIFI_PASSWORD must be empty or 8..63 characters (WPA2)")
|
|
endif()
|
|
foreach(_v NINLIGHT_WIFI_SSID NINLIGHT_WIFI_PASSWORD NINLIGHT_AP_PASSWORD NINLIGHT_TOKEN)
|
|
if("${${_v}}" MATCHES "[\"\\\\;]")
|
|
message(FATAL_ERROR "${_v} must not contain \", \\ or ;")
|
|
endif()
|
|
endforeach()
|
|
|
|
include(cmake/version.cmake)
|
|
include(cmake/sdk.cmake)
|
|
include(cmake/image.cmake)
|
|
|
|
set(NL_FW_SOURCES
|
|
src/app_main.c
|
|
src/util/nl_sha256.c
|
|
src/util/nl_crc.c
|
|
src/board/nl_board.c
|
|
src/light/nl_color.c
|
|
src/light/nl_engine.c
|
|
src/light/nl_control.c
|
|
src/light/nl_boot_state.c
|
|
src/config/nl_settings.c
|
|
src/config/nl_state_log.c
|
|
src/config/nl_reset_counter.c
|
|
src/config/nl_flash_w600.c
|
|
src/drivers/nl_led.c
|
|
src/drivers/nl_pwm_leds.c
|
|
src/drivers/nl_sm2135eh.c
|
|
src/sys/nl_console_cmd.c
|
|
src/sys/nl_console.c
|
|
src/sys/fwup_hook_default.c
|
|
src/sys/sdk_stubs.c
|
|
src/net/nl_names.c
|
|
src/net/nl_http.c
|
|
src/net/nl_api.c
|
|
src/net/nl_wifi_sm.c
|
|
src/net/nl_wifi_w600.c
|
|
src/net/nl_mdns.c
|
|
src/net/nl_http_server.c
|
|
src/sys/nl_app_api.c
|
|
third_party/cJSON/cJSON.c)
|
|
|
|
# C1: deeply nested JSON must not overflow the 6 KB HTTP task stack (parse_value recurses).
|
|
set_source_files_properties(third_party/cJSON/cJSON.c PROPERTIES
|
|
COMPILE_DEFINITIONS "CJSON_NESTING_LIMIT=8")
|
|
|
|
set(NL_FW ninlight_${NINLIGHT_DEVICE})
|
|
add_executable(${NL_FW} ${NL_SDK_BOOT_SOURCES} ${NL_FW_SOURCES} ${NL_SDK_OBJECTS})
|
|
set_target_properties(${NL_FW} PROPERTIES SUFFIX ".elf")
|
|
target_compile_definitions(${NL_FW} PRIVATE
|
|
NINLIGHT_DEVICE_ID="${NINLIGHT_DEVICE}"
|
|
NINLIGHT_FW_VERSION="${NINLIGHT_FW_VERSION}"
|
|
NINLIGHT_BUILD_DATE="${NINLIGHT_BUILD_DATE}"
|
|
NINLIGHT_TOKEN_DEFAULT="${NINLIGHT_TOKEN}"
|
|
NINLIGHT_WIFI_SSID="${NINLIGHT_WIFI_SSID}"
|
|
NINLIGHT_WIFI_PASSWORD="${NINLIGHT_WIFI_PASSWORD}"
|
|
NINLIGHT_AP_PASSWORD="${NINLIGHT_AP_PASSWORD}")
|
|
target_link_options(${NL_FW} PRIVATE
|
|
-static -nostartfiles -T${SDK_DIR}/ld/link_w600_2m.ld
|
|
-Wl,-Map=${CMAKE_BINARY_DIR}/${NL_FW}.map)
|
|
target_link_libraries(${NL_FW} PRIVATE
|
|
-Wl,--whole-archive ${SDK_DIR}/Lib/GNU/libwlan.a -Wl,--no-whole-archive m)
|
|
nl_add_images(${NL_FW})
|
|
|
|
message(STATUS "ninlight ${NINLIGHT_FW_VERSION} for ${NINLIGHT_DEVICE}")
|