Hello,
it seems that in dict_db, if DB->cursor() fails for any reason,
postfix would end up dereferencing a NULL pointer.
Maybe worth adding a check so that the error is logged at least.
Cheers,
Fedor
---
src/util/dict_db.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/util/dict_db.c b/src/util/dict_db.c
index 053db3c..4c714fa 100644
--- a/src/util/dict_db.c
+++ b/src/util/dict_db.c
@@ -446,8 +446,10 @@ static int dict_db_sequence(DICT *dict, int function,
*/
switch (function) {
case DICT_SEQ_FUN_FIRST:
- if (dict_db->cursor == 0)
- DICT_DB_CURSOR(db, &(dict_db->cursor));
+ if (dict_db->cursor == 0) {
+ if ((status = DICT_DB_CURSOR(db, &(dict_db->cursor))) != 0)
+ msg_fatal("error [%d] initializing cursor for %s: %m", status,
dict_db->dict.name);
+ }
db_function = DB_FIRST;
break;
case DICT_SEQ_FUN_NEXT:
--
2.53.0
_______________________________________________
Postfix-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
