1. Forum
  2. DOVE-Net
  3. Synchronet Programming

  • src/encode/lzh.c

    From Deucе@VERT to Git commit to main/sbbs/master on Thu Jan 2 10:50:20 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/bd91f791e7aad5f2626bb58e
    Modified Files:
    src/encode/lzh.c
    Log Message:
    Attempt to fix Coverity CID 454296

    This is that rare case where a Coverity fix makes the code more
    obvious and shorter.

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Fri Jan 3 23:30:36 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/be80773e436f62d5d927ea28
    Modified Files:
    src/encode/lzh.c
    Log Message:
    Add a (much) larger test vector for LZH

    It has a 4k window, we need to fill it multiple times.

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Sat Jan 4 16:19:54 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/bdc64cc9090f0d6825c4dcc5
    Modified Files:
    src/encode/lzh.c
    Log Message:
    Add a (much) larger test vector for LZH

    It has a 4k window, we need to fill it multiple times.

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Sun Jan 5 09:21:55 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/89d74a14f8cf6fdeae6984a0
    Modified Files:
    src/encode/lzh.c
    Log Message:
    More LZH updates

    - Make lzh_update() return a bool indicating success
    - Eliminated calculating array size, use macro
    - Fail if the current entry is not in the frequency array

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Sun Jan 5 09:37:11 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/8e14b84ef2708e8ccbb4d90d
    Modified Files:
    src/encode/lzh.c
    Log Message:
    What kind of monster puts a size_t after a uint8_t in a struct?

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Sun Jan 5 21:15:24 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/a57998b2da99464fd99ea375
    Modified Files:
    src/encode/lzh.c
    Log Message:
    Cap the search at LZH_ROOT (avoiding the sentry)

    This should take care of the last three overflow and truncation
    concerns that Coverity has.

    ---
    ￭ Synchronet ￭ Vertrauen ￭ Home of Synchronet ￭ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Mon Jan 6 06:48:49 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/82efb6413394432221bd6cbb
    Modified Files:
    src/encode/lzh.c
    Log Message:
    Suppress Coverity assertion.

    Coverity believes that after checking that c < LZH_ROOT (636), it's
    possible for (c + 1) to be greater than 65535.

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