[patch] actsync rejects hannover._wichtig

Julien ÉLIE julien at trigofacile.com
Fri Mar 19 19:58:13 UTC 2010


Hi Florian,

> You almost fooled me... or I fooled you. Lines 1142 to 1152 have to be
> looked at together, and what would need to be amended is of course line
> 1146 (isalpha()...) so that non_num will be set to true:

OK, now done.

--- backends/actsync.c  (révision 9001)
+++ backends/actsync.c  (copie de travail)
@@ -1140,12 +1138,13 @@
        return 1;
     }

-    /* must start with an alphanumeric ascii character */
+    /* must start with an ascii character */
     if (!isascii(name[0])) {
        return 1;
     }
     /* set non_num as needed */
-    if (isalpha((int)name[0])) {
+    if (isalpha((int)name[0])
+        || name[0] == '+' || name[0] == '-' || name[0] == '_') {
        non_num = true;
     } else if ((int)isdigit((int)name[0])) {
        non_num = false;



> I think it would be best to follow the actsync approach and have a
> global configuration variable num_chk or allownumericgroup or some such.
> And even if forbidden innd would still need to accept the article if
> it's crossposted to some other valid group, right?

Yes.

And more generally, actsync should be able to cope up with canonized
UTF-8 newsgroup names...



> What I meant to say is that I don't think the strspn() in that line has
> any effect in practice, and it's probably not meant that way - or am I
> overlooking something here?

It only forbids newsgroups whose name is a digit:

% ctlinnd newgroup 42
ctlinnd: Illegal newsgroup name

-- 
Julien ÉLIE

« E Roma nemo te diuidet omen amore. » 




More information about the inn-workers mailing list