Use container size rather than magic number 64

neutrinolabs · Sep 7, 2023 · 747c98f · 747c98f
1 parent d4bb793
commit 747c98f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/libxrdp/xrdp_caps.c b/libxrdp/xrdp_caps.c
@@ -564,7 +564,7 @@ xrdp_caps_process_codecs(struct xrdp_rdp *self, struct stream *s, int len)
             LOG(LOG_LEVEL_INFO, "xrdp_caps_process_codecs: NSCodec(%s), codec id [%d], properties len [%d]",
                 codec_guid_str, codec_id, codec_properties_length);
             self->client_info.ns_codec_id = codec_id;
-            i1 = MIN(64, codec_properties_length);
+            i1 = MIN(sizeof(self->client_info.ns_prop), codec_properties_length);
             g_memcpy(self->client_info.ns_prop, s->p, i1);
             self->client_info.ns_prop_len = i1;
         }
@@ -573,7 +573,7 @@ xrdp_caps_process_codecs(struct xrdp_rdp *self, struct stream *s, int len)
             LOG(LOG_LEVEL_INFO, "xrdp_caps_process_codecs: RemoteFX(%s), codec id [%d], properties len [%d]",
                 codec_guid_str, codec_id, codec_properties_length);
             self->client_info.rfx_codec_id = codec_id;
-            i1 = MIN(64, codec_properties_length);
+            i1 = MIN(sizeof(self->client_info.rfx_prop), codec_properties_length);
             g_memcpy(self->client_info.rfx_prop, s->p, i1);
             self->client_info.rfx_prop_len = i1;
         }
@@ -582,7 +582,7 @@ xrdp_caps_process_codecs(struct xrdp_rdp *self, struct stream *s, int len)
             LOG(LOG_LEVEL_INFO, "xrdp_caps_process_codecs: Image RemoteFX(%s), codec id [%d], properties len [%d]",
                 codec_guid_str, codec_id, codec_properties_length);
             self->client_info.irfx_codec_id = codec_id;
-            i1 = MIN(64, codec_properties_length);
+            i1 = MIN(sizeof(self->client_info.irfx_prop), codec_properties_length);
 
             g_memcpy(self->client_info.irfx_prop, s->p, i1);
             self->client_info.irfx_prop_len = i1;
@@ -609,7 +609,7 @@ xrdp_caps_process_codecs(struct xrdp_rdp *self, struct stream *s, int len)
             LOG(LOG_LEVEL_INFO, "xrdp_caps_process_codecs: H264(%s), codec id [%d], properties len [%d]",
                 codec_guid_str, codec_id, codec_properties_length);
             self->client_info.h264_codec_id = codec_id;
-            i1 = MIN(64, codec_properties_length);
+            i1 = MIN(sizeof(self->client_info.h264_prop), codec_properties_length);
             g_memcpy(self->client_info.h264_prop, s->p, i1);
             self->client_info.h264_prop_len = i1;
         }