• src/sbbs3/chksmb.c

    From rswindell@VERT to CVS commit on Wed Nov 16 02:32:51 2016
    src/sbbs3 chksmb.c 1.53 1.54
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/tmp/cvs-serv23252

    Modified Files:
    chksmb.c
    Log Message:
    Changed SMB_MSG_TYPE_VOTE to SMB_MSG_TYPE_BALLOT:
    There (will be) the possibilty to cast mutliple votes in a single vote-msg (ballot), when multi-choice polls are introduced later. There were just too many things call vote/votes and it was getting confusing in my own head. So the msg type for messages that contain votes is now "ballot".



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ telnet://vert.synchro.net
  • From rswindell@VERT to CVS commit on Wed Nov 23 04:41:40 2016
    src/sbbs3 chksmb.c 1.54 1.55
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Count missing Message-IDs only locally posted messages as an error.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ telnet://vert.synchro.net
  • From rswindell@VERT to CVS commit on Wed Nov 23 23:58:40 2016
    src/sbbs3 chksmb.c 1.55 1.56
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    No longer reports index/header value mismatches for indices of duplicate
    header records (index with the same message number points to a different message header).



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ telnet://vert.synchro.net
  • From rswindell@VERT to CVS commit on Thu Dec 1 18:22:12 2016
    src/sbbs3 chksmb.c 1.56 1.57
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Detect control characters a certain header fields (to/from/subj) and report
    as errors.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ telnet://vert.synchro.net
  • From rswindell@VERT to CVS commit on Thu Nov 16 03:17:09 2017
    src/sbbs3 chksmb.c 1.57 1.58
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Report errors if the header allocation (sha) or data allocation (sda)
    files don't correlate correctly with the data (sdt) or hdr (shd) files.
    Report errors if either the data or hdr file are not a logical length
    (e.g. not evenly divisble by the block length).
    Report the oldest message (in days) in the msgbase, from import date/time.

    ToDo: report messages posted/imported (or written) in the future
    ToDo: keep the stats in an array for sensible clearing and reporting



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Tue Feb 20 23:09:32 2018
    src/sbbs3 chksmb.c 1.58 1.59
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Changed 'fixsmb' suggestion to be less optomistic.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Wed Feb 21 02:42:19 2018
    src/sbbs3 chksmb.c 1.59 1.60
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Added an option ('-i') to not report missing Message IDs (it's normal
    for very old message bases).



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Sat Mar 3 02:10:33 2018
    src/sbbs3 chksmb.c 1.60 1.61
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Print all the details of a corrupted hash record.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Mon Mar 26 01:30:53 2018
    src/sbbs3 chksmb.c 1.61 1.62
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/tmp/cvs-serv27074

    Modified Files:
    chksmb.c
    Log Message:
    Resolve msvc warnings (and confirm that cvs commits are making it into
    syncprog and the commit log again).



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Tue Jul 17 02:42:36 2018
    src/sbbs3 chksmb.c 1.62 1.63
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/tmp/cvs-serv12858

    Modified Files:
    chksmb.c
    Log Message:
    Replace the hard-coded beep/BEL characters in the "Hit any key to continue..." prompts with the conditional beep enabled only with the '-b' option
    (silent by default). Thanks to Nelgin.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Fri Oct 5 05:24:46 2018
    src/sbbs3 chksmb.c 1.63 1.64
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Detect and report the message with the largest data (e.g. body text) size.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Fri Feb 1 07:49:16 2019
    src/sbbs3 chksmb.c 1.64 1.65
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/tmp/cvs-serv31941

    Modified Files:
    chksmb.c
    Log Message:
    Fix warnings raised by GCC v7.3.0:
    warning: ΓÇÿ*ΓÇÖ in boolean context, suggest ΓÇÿ&&ΓÇÖ instead
    note: ΓÇÿsprintfΓÇÖ output between 5 and 132 bytes into a destination of size 128



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Sat Feb 16 07:26:40 2019
    src/sbbs3 chksmb.c 1.65 1.66
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Detect and report all messages with overlapping headers.
    This has not actually been an observed problem, but I just realized while working on smb optimizations that this condition would not be reported by chksmb.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rampage@VERT/SESTAR to Digital Man on Sat Feb 16 16:22:23 2019
    Re: src/sbbs3/chksmb.c
    By: rswindell to CVS commit on Sat Feb 16 2019 02:26:40

    Modified Files:
    chksmb.c
    Log Message:
    Detect and report all messages with overlapping headers.

    i've been seeing something that may be similar with new files added to the system with addfiles (executed from tickit)... when i look at my files listings in the BBS, i'll see "strange" files marked as offline but their name is part of another file's description and their description is part of that other file's description... i have successfully removed those ""files"" from the listings by using the remove command within the BBS... the real files are not affected by this remove AFAICS... this is why i was asking about how to see offline files (on IRC) the other day... i have no idea where to look to try to figure out this problem...


    )\/(ark

    ---
    ■ Synchronet ■ SouthEast Star Mail HUB - SESTAR
  • From Rampage@VERT/SESTAR to Digital Man on Sat Feb 16 16:24:30 2019
    Re: src/sbbs3/chksmb.c
    By: Rampage to Digital Man on Sat Feb 16 2019 11:22:23

    i forgot to add that i do have terminal captures of what i've seen and done to fix the problem if you need to see them... not sure if they will help but i do have some...


    )\/(ark

    ---
    ■ Synchronet ■ SouthEast Star Mail HUB - SESTAR
  • From Digital Man@VERT to Rampage on Sat Feb 16 21:19:19 2019
    Re: src/sbbs3/chksmb.c
    By: Rampage to Digital Man on Sat Feb 16 2019 11:22 am

    Re: src/sbbs3/chksmb.c
    By: rswindell to CVS commit on Sat Feb 16 2019 02:26:40

    Modified Files:
    chksmb.c
    Log Message:
    Detect and report all messages with overlapping headers.

    i've been seeing something that may be similar with new files added to the system with addfiles (executed from tickit)... when i look at my files listings in the BBS, i'll see "strange" files marked as offline but their name is part of another file's description and their description is part of that other file's description... i have successfully removed those ""files"" from the listings by using the remove command within the BBS... the real files are not affected by this remove AFAICS... this is why i was asking about how to see offline files (on IRC) the other day... i have no idea where to look to try to figure out this problem...

    Whatever the issue, it's not related to SMB as the current filebase doesn't use SMB. If you figure out how to reproduce the problem or send me the data files (data/dirs/*) for the corrupt areas, I'll have a look-see.

    digital man

    This Is Spinal Tap quote #1:
    Nigel Tufnel: These go to eleven.
    Norco, CA WX: 56.2°F, 51.0% humidity, 4 mph N wind, 0.12 inches rain/24hrs

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Sun Feb 17 02:24:51 2019
    src/sbbs3 chksmb.c 1.66 1.67
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    If message's index offset exactly matches another message, that's already reported as "Duplicate index offset", so overlapping headers means the
    index offset must not be an exact duplicate, so we don't report the same corrupt message parameter (its offset into the header file) 2 different
    ways.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rampage@VERT/SESTAR to Digital Man on Sun Feb 17 17:53:21 2019
    Re: src/sbbs3/chksmb.c
    By: Digital Man to Rampage on Sat Feb 16 2019 16:19:19

    Whatever the issue, it's not related to SMB as the current
    filebase doesn't use SMB.

    yeah, i know it doesn't use SMB :) it was just the similarity of parts overlapping that drew my attention...

    If you figure out how to reproduce the problem or send me the
    data files (data/dirs/*) for the corrupt areas, I'll have a
    look-see.

    i hadn't thought about saving the data files before cleaning them up... i'll keep an eye out and grab them the next time i see the problem and get them to you...


    )\/(ark

    ---
    ■ Synchronet ■ SouthEast Star Mail HUB - SESTAR
  • From mark lewis@VERT to Digital Man on Mon Feb 18 17:53:10 2019
    On 2019 Feb 16 16:19:18, you wrote to Rampage:

    but their name is part of another file's description and their
    description is part of that other file's description... i have
    successfully removed those ""files"" from the listings by using the
    remove command within the BBS... the real files are not affected by
    this remove AFAICS... this is why i was asking about how to see offline
    files (on IRC) the other day... i have no idea where to look to try to
    figure out this problem...

    Whatever the issue, it's not related to SMB as the current filebase doesn't use SMB. If you figure out how to reproduce the problem or
    send me the data files (data/dirs/*) for the corrupt areas, I'll have
    a look-see.

    i don't know if you saw it or not but i did send you an email sunday evening with two of my files bases' database files zipped and two screenshots of the problem... one shot for each area...

    FWIW: the only things touching tho filebases are addfiles (via tickit) and sbbs... nothing 3rd party or self-written...

    )\/(ark

    Always Mount a Scratch Monkey
    Do you manage your own servers? If you are not running an IDS/IPS yer doin' it wrong...
    ... When man builds better mousetraps, nature builds better mice.
    ---
    * Origin: (1:3634/12.73)
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Digital Man@VERT to mark lewis on Mon Feb 18 19:11:15 2019
    Re: src/sbbs3/chksmb.c
    By: mark lewis to Digital Man on Mon Feb 18 2019 12:53 pm


    On 2019 Feb 16 16:19:18, you wrote to Rampage:

    but their name is part of another file's description and their
    description is part of that other file's description... i have
    successfully removed those ""files"" from the listings by using the
    remove command within the BBS... the real files are not affected by
    this remove AFAICS... this is why i was asking about how to see offline
    files (on IRC) the other day... i have no idea where to look to try to
    figure out this problem...

    Whatever the issue, it's not related to SMB as the current filebase doesn't use SMB. If you figure out how to reproduce the problem or
    send me the data files (data/dirs/*) for the corrupt areas, I'll have
    a look-see.

    i don't know if you saw it or not but i did send you an email sunday evening with two of my files bases' database files zipped and two screenshots of the problem... one shot for each area...

    FWIW: the only things touching tho filebases are addfiles (via tickit) and sbbs... nothing 3rd party or self-written...

    I saw and saved the email. I haven't looked at it yet. Most likely it's an addfiles bug. Do you have the input file to addfiles and the command-line that may have caused the corruption?

    digital man

    Synchronet/BBS Terminology Definition #32:
    FTSC = FidoNet Technical Standards Committee
    Norco, CA WX: 55.0°F, 35.0% humidity, 5 mph SSW wind, 0.00 inches rain/24hrs

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From mark lewis@VERT to Digital Man on Tue Feb 19 02:13:54 2019
    On 2019 Feb 18 14:11:14, you wrote to me:

    FWIW: the only things touching tho filebases are addfiles (via tickit)
    and sbbs... nothing 3rd party or self-written...

    I saw and saved the email. I haven't looked at it yet. Most likely
    it's an addfiles bug.

    ok...

    Do you have the input file to addfiles and the command-line that may
    have caused the corruption?

    i can dig out the command line from my logs but tickit deletes the files.bbs file it creates when it is finished... it actually writes a new one for each file being imported...

    it is actually a pretty generic command... only the area ID (and uploader name in my case) changes each time while the rest of the line stays the same... here's one of the 5 or 6 executed today...

    Feb 18 16:22:07 sestar sbbs: evnt TICKIT Executing: '/sbbs/exec/addfiles fdist-xofchubs -x "XOFCHUBSLST File Dist" -zd +/sbbs/temp/event/tickit-files.bbs 24 13
    /sbbs/data/addfiles.log 2>&1'.

    none of the areas processed today show any corruption at this time...

    )\/(ark

    Always Mount a Scratch Monkey
    Do you manage your own servers? If you are not running an IDS/IPS yer doin' it wrong...
    ... My hobbies include closing the elevator door before someone else gets on. ---
    * Origin: (1:3634/12.73)
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Tue Sep 24 03:41:19 2019
    src/sbbs3 chksmb.c 1.69 1.70
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Fix the bug introduced into this copy/pasted version of ultoac
    (unsigned-long to ASCII with commas) in rev 1.68 (addressing warnings).
    A size_t (i) can't go negative. Duh.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Thu Apr 2 14:40:32 2020
    src/sbbs3 chksmb.c 1.70 1.71
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/tmp/cvs-serv11786

    Modified Files:
    chksmb.c
    Log Message:
    Only warn about an invalid hash record timestamp if the time is more than an hour in the future. This allows for small adjustments in the local system clock.


    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Sat Apr 4 17:36:38 2020
    src/sbbs3 chksmb.c 1.71 1.72
    Update of /cvsroot/sbbs/src/sbbs3
    In directory cvs:/home/rswindell/sbbs/src/sbbs3

    Modified Files:
    chksmb.c
    Log Message:
    Increase the "invalid length" of a hashed data record from 1MB to 1GB. Apparently some USENET articles are >= 1MB these days. Thanks Coz.



    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to sbbs/master on Sat Sep 26 17:38:58 2020
    https://gitlab.synchro.net/sbbs/sbbs/-/commit/17b7aee1b051c3b366c5dfa6
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Use smb_msg_type() to double-check the header type against attribute flags.

    The inferred message type should match the stored message type. Confirm that.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Sun Apr 25 19:36:21 2021
    https://gitlab.synchro.net/main/sbbs/-/commit/35f764eef674aaeb7ce528b2
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Always report the total LZH compressed messages

    This looks like a 20-year-old copy/paste bug. CID 33168.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Tue Mar 1 22:43:08 2022
    https://gitlab.synchro.net/main/sbbs/-/commit/faa31747a860b290922393c3
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Zero-initialize smb before using

    Addresses CID 350318

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Sat Jun 4 20:26:40 2022
    https://gitlab.synchro.net/main/sbbs/-/commit/3ccd8ef8b20ce61a37fe4d5a
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Fix copy/paste bug discovered/reported by gcc version 11.2.0

    ... in debug print statement (MD5 hast decoded/printed as SHA-1 hash).

    Reported by Nelgin running gcc version 11.2.0 (Ubuntu 11.2.0-19ubuntu1)

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Mon May 22 16:41:53 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/2f4bebb1d0e9e28977285cca
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Support new -S option to not check Subject CRCs

    the Subject CRC calculation was changed in May of 2022 (trailing whitespace is removed prior to calculation), so messages imported before this date/revision of sbbs might be detected as having an invalid/mismatch Subject CRC. This -S option can be used to suppress such errors (for Keyop).

    Don't support /option syntax any longer in non-*nix builds (just -options). Options are now case-sensitive (i.e. -S and -s are different options).
    Bump version to 3.20.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Fri May 26 17:14:21 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/d9a5999debfdc1039d39626e
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Add -N option to disable to/from name CRC checking

    Recently started truncating trailing whitespace from sender/recipient names before calculating the CRC value (similar to what we do for message subjects
    as of a year ago). To suppress errors when evaluating messages (with trailing whitespace in sender or recipient namems) created or imported before this change, use this new -N option.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on ChromeOS)@VERT to Git commit to main/sbbs/master on Sun Jul 9 20:23:13 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/86825b66e08061e8675591ab
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Only truncate ".anything" from supplied msgbase path if it's ".shd"

    Make behavior consistent with smbutil and fixmsb.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows)@VERT to Git commit to main/sbbs/master on Mon Sep 25 17:39:43 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/c9c90ef7186e7ecd1103bc1b
    Modified Files:
    src/sbbs3/chksmb.c
    Log Message:
    Replace many references to "Message" with "File" when scanning a file base

    This could use more cleanup (e.g. instances of "MSGERR:") and in the summary report there are still mentions of "message" after scanning a file base.

    ---
    ■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net