michaelh/zephyr
1
0
Fork 0
Code Releases Activity

samples: modules: nanopb: fix encode buffer size

Browse source 
A bug was introduced when encoding/decoding moved to separate
functions where sizeof no longer had the correct argument.

The size of the encode buffer is passed to the encode function.

Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
This commit is contained in:
Pieter De Gendt 2021-04-12 13:11:41 +02:00 • committed by Maureen Helm
commit 40c25b931d
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
samples/modules/nanopb/src/main.c
View file

@ -12,7 +12,7 @@
#include <pb_decode.h> #include <pb_decode.h>
#include "src/simple.pb.h" #include "src/simple.pb.h"
bool encode_message(uint8_t *buffer, size_t *message_length) bool encode_message(uint8_t *buffer, size_t buffer_size, size_t *message_length)
{ {
bool status; bool status;
@ -26,7 +26,7 @@ bool encode_message(uint8_t *buffer, size_t *message_length)
SimpleMessage message = SimpleMessage_init_zero; SimpleMessage message = SimpleMessage_init_zero;
/* Create a stream that will write to our buffer. */ /* Create a stream that will write to our buffer. */
pb_ostream_t stream = pb_ostream_from_buffer(buffer, sizeof(buffer)); pb_ostream_t stream = pb_ostream_from_buffer(buffer, buffer_size);
/* Fill in the lucky number */ /* Fill in the lucky number */
message.lucky_number = 13; message.lucky_number = 13;
@ -73,7 +73,7 @@ void main(void)
size_t message_length; size_t message_length;
/* Encode our message */ /* Encode our message */
if (!encode_message(buffer, &message_length)) { if (!encode_message(buffer, sizeof(buffer), &message_length)) {
return; return;
} }