net: lwm2m: Fix time resource validation (558e2a55) · Commits · Wenxi XU / zephyr

When writing to a time resource, the logic was incorrect when validation was enabled for the resource - when determining the actual underlying data type we should compare against the data buffer length and not the validation buffer length. Signed-off-by:

Robert Lubos <robert.lubos@nordicsemi.no>

subsys/net/lib/lwm2m/lwm2m_message_handling.c

+3 −3

Changes for subsys/net/lib/lwm2m/lwm2m_message_handling.c: 3 added lines, 3 removed lines.
Original line number	Diff line number	Diff line
		@@ -1163,14 +1163,14 @@ int lwm2m_write_handler(struct lwm2m_engine_obj_inst *obj_inst, struct lwm2m_eng
		break;
		}

		if (write_buf_len == sizeof(time_t)) {
		if (data_len == sizeof(time_t)) {
		(time_t )write_buf = temp_time;
		len = sizeof(time_t);
		} else if (write_buf_len == sizeof(uint32_t)) {
		} else if (data_len == sizeof(uint32_t)) {
		(uint32_t )write_buf = (uint32_t)temp_time;
		len = sizeof(uint32_t);
		} else {
		LOG_ERR("Time resource buf len not supported %zu", write_buf_len);
		LOG_ERR("Time resource buf len not supported %zu", data_len);
		ret = -EINVAL;
		}

Admin message