ams: copy the payload struct size into the message slot

The slot copy length came from a macro that adds the message length, which over-read past the payload struct since the message data is referenced by pointer, not stored inline. Copy exactly the struct size, which is what the slot stores and the consumer reads back.

Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>

Author: lrgirdwo

src/lib/ams.c

@@ -285,9 +285,14 @@ static uint32_t ams_push_slot(struct ams_shared_context __sparse_cache *ctx_shar
 
 	for (uint32_t i = 0; i < ARRAY_SIZE(ctx_shared->slots); ++i) {
 		if (ctx_shared->slot_uses[i] == 0) {
+			/* the slot stores the payload struct (read back via
+			 * u.msg); AMS_MESSAGE_SIZE() adds message_length, which
+			 * over-reads past the struct since message is a pointer,
+			 * not inline data
+			 */
 			err = memcpy_s((__sparse_force void *)ctx_shared->slots[i].u.msg_raw,
 				       sizeof(ctx_shared->slots[i].u.msg_raw),
-				       msg, AMS_MESSAGE_SIZE(msg));
+				       msg, sizeof(*msg));
 
 			if (err != 0)
 				return AMS_INVALID_SLOT;