The project "Citadel server, clients, utilities":

    The branch, master has been updated
       via  e67c92e0be75acaf0e95dc1c1764d69f8c67328d (commit)
      from  ac768c83b658bf13adf61072d094be1c652451f5 (commit)


- Log -----------------------------------------------------------------
commit e67c92e0be75acaf0e95dc1c1764d69f8c67328d
Author: Harlow Solutions <[email protected]>
Date:   Wed Aug 27 22:41:05 2025 -0400

Clean up CtdlFetchMsgList() processing:

room_ops.c
CtdlFetchMsgList() says it will return NULL list for zero messages.
Original code returned a dummy list if DB fetch returned NULL data,
which is not consistent with comments.
Corrected to return NULL.

All other source changes were to places that called CtdlFetchMsgList()
to handle zero lists properly (don’t free() NULL and to not access list if 
empty).
Also, some functions were handling negative counts as an error.  Function never 
returns
negative, so changed handling to only handle empty lists and lists with 
messages.

-----------------------------------------------------------------------

Summary of changes:
 citadel/server/control.c                           |  2 +-
 citadel/server/euidindex.c                         |  6 +--
 .../modules/autocompletion/serv_autocompletion.c   |  6 ++-
 citadel/server/modules/expire/serv_expire.c        |  3 +-
 citadel/server/modules/imap/serv_imap.c            |  4 +-
 citadel/server/modules/nntp/serv_nntp.c            |  6 +--
 citadel/server/msgbase.c                           | 45 +++++++++++++---------
 citadel/server/room_ops.c                          | 12 ++----
 citadel/server/user_ops.c                          |  4 +-
 9 files changed, 46 insertions(+), 42 deletions(-)

-- 
Citadel server, clients, utilities

Reply via email to