[Users] [Bug 3939] New: XHDR newsgroups command endless loop

[noreply at thewildbeast.co.uk]

http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3939

            Bug ID: 3939
           Summary: XHDR newsgroups command endless loop
    Classification: Unclassified
           Product: Claws Mail
           Version: 3.16.0
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P3
         Component: NNTP
          Assignee: users at lists.claws-mail.org
          Reporter: jean.diraison at ac-rennes.fr

Created attachment 1829
  -->
http://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1829&action=edit
Manage -1 end special value in nntp_threaded_xhdr nntp_threaded_over

The XHDR newsgroups command keeps downloading data forever in some case.

This happens when nntp_threaded_xhdr() function [src/etpan/nntp-thread.c] is
called with "end" parameter equal to 0xffffffff = (guint32)-1. The condition in
the while loop is always true (cbeg <= end && cend <= end).

I have experienced this problem with groups where last article has been deleted
(for example server: richelieu.ac-versailles.fr:119  group:
ac-versailles.vie-du-reseau.discussions ).

When this occurs, the XOVER command returns no data. So "newlist" stay NULL in
news_get_msginfos_for_range() function [src/news.c]. In this case, the "first"
and "last" signed integers are equals to -1 in news_get_extra_fields()
function, and they are converted to unsigned 32 bits integers 0xffffffff in
nntp_threaded_xhdr() call [src/etpan/nntp-thread.c].



I don't know what is the best way to solve this problem:
* return from news_get_extra_fields() if msglist is NULL
* or manage -1 special case values in nntp_threaded_xhdr() to keep previous
behavior (by comparing values + ones)

-- 
You are receiving this mail because:
You are the assignee for the bug.