Remove templating from cf_pair_find().

This breaks the server, so the next commit will add templates
back in again, in a better fashion.

We also remove the ability to do cf_pair_find(cs, NULL), and have
it return the first CONF_PAIR.  After searching the code, no one
uses this functionality.  So it's best to remove it.

src/main/conffile.c

@@ -2010,33 +2010,12 @@ void cf_file_free(CONF_SECTION *cs)
  */
 CONF_PAIR *cf_pair_find(CONF_SECTION const *cs, char const *name)
 {
-	CONF_ITEM	*ci;
-	CONF_PAIR	*cp = NULL;
-
-	if (!cs) return NULL;
-
-	/*
-	 *	Find the name in the tree, for speed.
-	 */
-	if (name) {
-		CONF_PAIR mycp;
+	CONF_PAIR mycp;
 
-		mycp.attr = name;
-		cp = rbtree_finddata(cs->pair_tree, &mycp);
-	} else {
-		/*
-		 *	Else find the first one that matches
-		 */
-		for (ci = cs->children; ci; ci = ci->next) {
-			if (ci->type == CONF_ITEM_PAIR) {
-				return cf_itemtopair(ci);
-			}
-		}
-	}
-
-	if (cp || !cs->template) return cp;
+	if (!cs || !name) return NULL;
 
-	return cf_pair_find(cs->template, name);
+	mycp.attr = name;
+	return rbtree_finddata(cs->pair_tree, &mycp);
 }
 
 /*