]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#3770] Revert length changes in mysql host code
authorThomas Markwalder <tmark@isc.org>
Mon, 21 Jul 2025 18:44:25 +0000 (14:44 -0400)
committerThomas Markwalder <tmark@isc.org>
Mon, 21 Jul 2025 18:44:25 +0000 (14:44 -0400)
modified:   src/hooks/dhcp/mysql/mysql_host_data_source.cc

src/hooks/dhcp/mysql/mysql_host_data_source.cc

index 92441661a0cc32025c0745b0d854cfed69c3e2ea..6644671d73579145c6a3d7f512bee8bb222ad45a 100644 (file)
@@ -619,7 +619,7 @@ public:
         // Convert user_context to string as well.
         std::string user_context;
         if (user_context_null_ == MLM_FALSE) {
-            user_context_[user_context_length_ - 1] = '\0';
+            user_context_[user_context_length_] = '\0';
             user_context.assign(user_context_);
         }
 
@@ -949,7 +949,7 @@ private:
             if (space_null_ == MLM_FALSE) {
                 // Typically, the string values returned by the database are not
                 // NULL terminated.
-                space_[space_length_ - 1] = '\0';
+                space_[space_length_] = '\0';
                 space.assign(space_);
             }
 
@@ -962,21 +962,21 @@ private:
             // Convert formatted_value to string.
             std::string formatted_value;
             if (formatted_value_null_ == MLM_FALSE) {
-                formatted_value_[formatted_value_length_ - 1] = '\0';
+                formatted_value_[formatted_value_length_] = '\0';
                 formatted_value.assign(formatted_value_);
             }
 
             // Convert user_context to string.
             std::string user_context;
             if (user_context_null_ == MLM_FALSE) {
-                user_context_[user_context_length_ - 1] = '\0';
+                user_context_[user_context_length_] = '\0';
                 user_context.assign(user_context_);
             }
 
             // Convert client classes to string.
             std::string client_classes;
             if (client_classes_null_ == MLM_FALSE) {
-                client_classes_[client_classes_length_ - 1] = '\0';
+                client_classes_[client_classes_length_] = '\0';
                 client_classes.assign(client_classes_);
             }