kozorizki/haproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MINOR: stktable: provide an unchecked version of stktable_data_ptr()

Browse Source 
stktable_data_ptr() currently performs null pointer checks but most
callers don't check the result since they know by construction that
it cannot be null. This causes valid warnings when building with
-Wextra which are worth addressing since it will result in better
code. Let's provide an unguarded version of this function for use
where the check is known to be useless and untested.
This commit is contained in:
Willy Tarreau 2018-09-20 11:06:33 +02:00
parent c2b7f80a91
commit babc15e8cf
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
include/proto/stick_table.h
View File

@ -113,6 +113,14 @@ static inline int stktable_alloc_data_type(struct stktable *t, int type, const c
return PE_NONE;
}
/* return pointer for data type <type> in sticky session <ts> of table <t>, all
* of which must exist (otherwise use stktable_data_ptr() if unsure).
*/
static inline void *__stktable_data_ptr(struct stktable *t, struct stksess *ts, int type)
{
return (void *)ts + t->data_ofs[type];
}
/* return pointer for data type <type> in sticky session <ts> of table <t>, or
* NULL if either <ts> is NULL or the type is not stored.
*/
@ -127,7 +135,7 @@ static inline void *stktable_data_ptr(struct stktable *t, struct stksess *ts, in
if (!ts)
return NULL;
return (void *)ts + t->data_ofs[type];
return __stktable_data_ptr(t, ts, type);
}
/* kill an entry if it's expired and its ref_cnt is zero */