File pacemaker#3292-0004-Low-scheduler-restore-nvpair-behavior-without-id-ref.patch of Package pacemaker.34783

From 4d0685e2b789002bdd3090c52e76e45b0c7eb504 Mon Sep 17 00:00:00 2001
From: Ken Gaillot <kgaillot@redhat.com>
Date: Thu, 14 Dec 2023 14:16:32 -0600
Subject: [PATCH] Low: scheduler: restore nvpair behavior without id-ref

... per review
---
 lib/pengine/rules.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

Index: pacemaker-2.0.4+20200616.2deceaa3a/lib/pengine/rules.c
===================================================================
--- pacemaker-2.0.4+20200616.2deceaa3a.orig/lib/pengine/rules.c
+++ pacemaker-2.0.4+20200616.2deceaa3a/lib/pengine/rules.c
@@ -760,18 +760,14 @@ populate_hash(xmlNode * nvpair_list, GHa
         if (crm_str_eq((const char *)an_attr->name, XML_CIB_TAG_NVPAIR, TRUE)) {
             xmlNode *ref_nvpair = expand_idref(an_attr, top);
 
-            if (ref_nvpair == NULL) {
-                continue; // Not possible with schema validation enabled
-            }
-
             name = crm_element_value(an_attr, XML_NVPAIR_ATTR_NAME);
-            if (name == NULL) {
+            if ((name == NULL) && (ref_nvpair != NULL)) {
                 name = crm_element_value(ref_nvpair, XML_NVPAIR_ATTR_NAME);
             }
 
             crm_trace("Setting attribute: %s", name);
             value = crm_element_value(an_attr, XML_NVPAIR_ATTR_VALUE);
-            if (value == NULL) {
+            if ((value == NULL) && (ref_nvpair != NULL)) {
                 value = crm_element_value(ref_nvpair, XML_NVPAIR_ATTR_VALUE);
             }
 
openSUSE Build Service is sponsored by