Skip to content
Snippets Groups Projects
Commit b846192f authored by Deucе's avatar Deucе :ok_hand_tone4:
Browse files

Just parse VER using /^VER (.*) ([^ ]*?)$/ 

This restores the meaning of remote_ver and still has a hack for
broken, unfixable mailers that don't advertise protocol v1.1
correctly (ie: Irex)
parent 10d7a690
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Hide whitespace changes
Inline Side-by-side
Showing
with 7 additions and 5 deletions
  • Rob Swindell :speech_balloon: @rswindell ·
    Owner

    Out of curiosity, I looked up how BinkD does the protocol version parsing:

    #define PRTCLNAME "binkp"
...
  if (!memcmp (s, "VER ", 4) &&
      (a = strstr (s, PRTCLNAME "/")) != 0 &&
      (b = strstr (a, ".")) != 0)
  {
    state->major = atoi (a + 6);
    state->minor = atoi (b + 1);
    Log (6, "remote uses " PRTCLNAME " v.%i.%i", state->major, state->minor);

    So... that would match "binkp/1.1" anywhere in the VER string (even in parenthesis), likely explaining why IRex never had a problem with BinkD in this regard.

0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment