pmxcfs: cleanup remaining_size calculation

using an end-pointer it's a bit more readable and gets rid
of an (int) cast

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

diff --git a/data/src/status.c b/data/src/status.c
index aab922c4ac4cedb9639904fb458dc94e753ae72b..62eaa76c4d08a2f919aec03c780b122da455704a 100644 (file)
--- a/data/src/status.c
+++ b/data/src/status.c
@@ -803,8 +803,9 @@ cfs_create_vmlist_msg(GString *str)
 static char *
 _get_property_value(char *conf, int conf_size, const char *prop, int prop_len)
 {
+       const char *const conf_end = conf + conf_size;
        char *line = conf;
-       int remaining_size;
+       size_t remaining_size;
 
        char *next_newline = memchr(conf, '\n', conf_size);
        if (next_newline == NULL) {
@@ -838,11 +839,11 @@ _get_property_value(char *conf, int conf_size, const char *prop, int prop_len)
                        return v_start;
                }
 next:
-               remaining_size = conf_size - (int) (next_newline - conf);
-               if (remaining_size <= 1 || remaining_size <= prop_len) {
+               line = next_newline + 1;
+               remaining_size = conf_end - line;
+               if (remaining_size <= prop_len) {
                        return NULL;
                }
-               line = next_newline + 1;
                next_newline = memchr(line, '\n', remaining_size);
                if (next_newline == NULL) {
                        return NULL; // valid property lines end with \n, but none in the config