zcore: heed HIDE_FROM_AL flag within function load_list_members

References: DESK-3373, g-web/321

Author: jengelh

exch/zcore/container_object.cpp

@@ -886,8 +886,14 @@ BOOL container_object::query_user_table(const PROPTAG_ARRAY *pproptags,
 			for (size_t i = first_pos; i < first_pos+row_count &&
 			     i < pcontainer->contents.pminid_array->count; ++i) {
 				ab_tree::ab_node node(pbase, pcontainer->contents.pminid_array->pl[i]);
-				if (!node.exists() || node.hidden() & AB_HIDE_FROM_AL)
+				if (!node.exists())
 					continue;
+				/*
+				 * No testing for HIDE* here, because
+				 * pminid_array is already prefiltered (when it
+				 * was made; ab_tree_match_minids is called by
+				 * load_user_table).
+				 */
 				pset->pparray[pset->count] = cu_alloc<TPROPVAL_ARRAY>();
 				if (pset->pparray[pset->count] == nullptr)
 					return FALSE;

exch/zcore/user_object.cpp

@@ -216,7 +216,7 @@ ec_error_t user_object::load_list_members(const RESTRICTION *res) try
 		auto mid = ab_tree::minid(ab_tree::minid::address, user_id);
 		node = {base, mid};
 		LONG_ARRAY unused{};
-		if (!node.exists() ||
+		if (!node.exists() || node.hidden() & AB_HIDE_FROM_AL ||
 		    !ab_tree_match_minids(base.get(), mid, res, &unused))
 			continue;
 		free(unused.pl);