summaryrefslogtreecommitdiff
path: root/net/ser/files/patch-modules::auth_db::authorize.c
blob: 15fb53b5e220888c9f67cd9a318cac5e9bb79556 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
$FreeBSD$

--- modules/auth_db/authorize.c
+++ modules/auth_db/authorize.c
@@ -101,7 +101,7 @@
 	}
 	pkg_free(col);
 
-	if ((RES_ROW_N(*res) == 0) || VAL_NULL(ROW_VALUES(RES_ROWS(*res)))) {
+	if (RES_ROW_N(*res) == 0) {
 		DBG("get_ha1(): no result for user \'%.*s@%.*s\'\n",
 		    _username->user.len, ZSW(_username->user.s), (use_domain ? (_domain->len) : 0), ZSW(_domain->s));
 		return 1;
@@ -212,9 +212,7 @@
 		rpid.s = NULL;
 		rpid.len = 0;
 		for (i = 0; i < avps_str_n; i++) {
-			if (avps_str[i].len != 4
-					|| VAL_NULL(&(result->rows[0].values[1 + avps_int_n + i]))
-					|| memcmp(avps_str[i].s, "rpid", 4) != 0)
+			if (avps_str[i].len != 4 || memcmp(avps_str[i].s, "rpid", 4) != 0)
 				continue;
 			rpid.s = (char*)VAL_STRING(&(result->rows[0].values[1 + avps_int_n + i]));
 			if(rpid.s!=NULL)
@@ -230,8 +228,6 @@
 			return -1;
 		case AUTHORIZED:
 			for (i = 0; i < avps_int_n; i++) {
-				if(VAL_NULL(&(result->rows[0].values[1 + i])))
-					continue;
 				iname.s = &(avps_int[i]);
 				ivalue.n = VAL_INT(&(result->rows[0].values[1 + i]));
 				add_avp(AVP_NAME_STR, iname, ivalue);
@@ -240,8 +236,7 @@
 			}
 			for (i = 0; i < avps_str_n; i++) {
 				value.s = (char*)VAL_STRING(&(result->rows[0].values[1 + avps_int_n + i]));
-				if(VAL_NULL(&(result->rows[0].values[1 + avps_int_n + i]))
-						|| value.s==NULL)
+				if(value.s==NULL)
 					continue;
 				iname.s = &(avps_str[i]);
 				value.len = strlen(value.s);