SubGhz Honeywell 5834: size encoder buffer for 48-bit frame only

Allocate encoder upload buffer for exactly 99 LevelDuration entries
(sync 3 + 48 bits * 2) instead of 128, saving ~116 bytes RAM per
encoder instance on Flipper Zero. Add H5834_UPLOAD_MAX constant and
use it for allocation and bounds check.

Author: herbenderbler

lib/subghz/protocols/honeywell_5834.c

@@ -32,6 +32,8 @@
 #define H5834_SYNC_PULSE    300
 #define H5834_GAP           120
 #define H5834_MIN_COUNT_BIT 48
+/* Encoder: sync(3) + 48 data bits * 2 (pulse+gap) = 99 LevelDuration entries */
+#define H5834_UPLOAD_MAX    (3 + (H5834_MIN_COUNT_BIT * 2))
 
 static const SubGhzBlockConst subghz_protocol_honeywell_5834_const = {
     .te_short = H5834_TE_SHORT,
@@ -105,7 +107,7 @@ void* subghz_protocol_encoder_honeywell_5834_alloc(SubGhzEnvironment* environmen
     instance->generic.protocol_name = instance->base.protocol->name;
 
     instance->encoder.repeat = 10;
-    instance->encoder.size_upload = 128;
+    instance->encoder.size_upload = H5834_UPLOAD_MAX;
     instance->encoder.upload = malloc(instance->encoder.size_upload * sizeof(LevelDuration));
     instance->encoder.is_running = false;
     return instance;
@@ -126,10 +128,10 @@ static bool subghz_protocol_encoder_honeywell_5834_get_upload(
     SubGhzProtocolEncoderHoneywell5834* instance) {
     furi_assert(instance);
 
-    /* sync(3) + data_bits * 2 = 3 + 48*2 = 99 max, well within 128 */
+    /* sync(3) + data_bits * 2; 48-bit frame => 99 entries (H5834_UPLOAD_MAX) */
     size_t index = 0;
     size_t size_upload = 3 + (instance->generic.data_count_bit * 2);
-    if(size_upload > instance->encoder.size_upload) {
+    if(size_upload > H5834_UPLOAD_MAX) {
         FURI_LOG_E(TAG, "Size upload exceeds allocated encoder buffer.");
         return false;
     }