• src/xpdev/xpprintf.c

    From deuce@VERT to CVS commit on Saturday, October 12, 2013 23:12:22
    src/xpdev xpprintf.c 1.37 1.38
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv9778

    Modified Files:
    xpprintf.c
    Log Message:
    Remove those most NOPy of NOPcodes... i=i and friends.
    Clang warnings.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Wednesday, February 05, 2014 02:22:17
    src/xpdev xpprintf.c 1.38 1.39
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv9476

    Modified Files:
    xpprintf.c
    Log Message:
    Fix Clang warnings... cast values to intptr_t before casting to a void *



    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Wednesday, February 05, 2014 19:13:53
    src/xpdev xpprintf.c 1.39 1.40
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv4646

    Modified Files:
    xpprintf.c
    Log Message:
    Use uintptr_t for unsigned casts.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Wednesday, April 23, 2014 22:46:55
    src/xpdev xpprintf.c 1.41 1.42
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv30550

    Modified Files:
    xpprintf.c
    Log Message:
    REAL BUG FOUND BY Coverity!

    xp_printf() interpreted the 'l' modifier as 'h' and the 'll' modifier as the (invalid) 'hl' modifier.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Thursday, April 24, 2014 00:02:59
    src/xpdev xpprintf.c 1.42 1.43
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31302

    Modified Files:
    xpprintf.c
    Log Message:
    Coverity bugs (multiple) Initialize temporary values to 0/NULL in xp_asprintf_next() as protection against an invalid type being passed in.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Thursday, April 24, 2014 00:06:00
    src/xpdev xpprintf.c 1.43 1.44
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31334

    Modified Files:
    xpprintf.c
    Log Message:
    Coverity bug: Return 0 from xp_printf_get_type() when an invalid specifier
    is detected, rather than an uninitialized value!




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Thursday, April 24, 2014 00:52:51
    src/xpdev xpprintf.c 1.44 1.45
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31864

    Modified Files:
    xpprintf.c
    Log Message:
    Fix the same l->h bug in xp_printf_get_type()




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Thursday, April 24, 2014 00:58:06
    src/xpdev xpprintf.c 1.45 1.46
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31983

    Modified Files:
    xpprintf.c
    Log Message:
    Fix missed temporary variable 'i' from last commit.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Tuesday, April 29, 2014 02:24:39
    src/xpdev xpprintf.c 1.46 1.47
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv12712

    Modified Files:
    xpprintf.c
    Log Message:
    Use genwrap.h instead of malloc.h (win32 only) for alloca()




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, October 06, 2014 14:45:36
    src/xpdev xpprintf.c 1.47 1.48
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv6498

    Modified Files:
    xpprintf.c
    Log Message:
    Typo in a comment... per Ragnorok




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Sunday, September 27, 2015 23:05:20
    src/xpdev xpprintf.c 1.48 1.49
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv17479

    Modified Files:
    xpprintf.c
    Log Message:
    If sprintf() returns a negative number, replace the template with "<error>".




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Sunday, September 27, 2015 23:09:56
    src/xpdev xpprintf.c 1.49 1.50
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv17745

    Modified Files:
    xpprintf.c
    Log Message:
    Fix typo in last commit.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 13:26:51
    src/xpdev xpprintf.c 1.50 1.51
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31335

    Modified Files:
    xpprintf.c
    Log Message:
    Add an asprintf() implementation for Win32. Not tested yet, but it should work if I can reuse a va_list argument.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 13:31:48
    src/xpdev xpprintf.c 1.51 1.52
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31478

    Modified Files:
    xpprintf.c
    Log Message:
    Use vsprintf(), not sprintf(), make a copy of the va_list, return the correct value for the Win32 asprintf().




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 13:34:54
    src/xpdev xpprintf.c 1.52 1.53
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv31577

    Modified Files:
    xpprintf.c
    Log Message:
    Remove MAX_ARG_LEN usage.
    Rename entry_buf to int_buf since it only ever holds an integer now.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 17:44:20
    src/xpdev xpprintf.c 1.54 1.55
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv16668

    Modified Files:
    xpprintf.c
    Log Message:
    Fix off-by-one in memmove() calculation which caused a crash on Linux when
    the expanded specified with or precision was more than one digit long.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 17:47:48
    src/xpdev xpprintf.c 1.55 1.56
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv17434

    Modified Files:
    xpprintf.c
    Log Message:
    Fix warnings. We no longer need to track width and precision since we have asprintf() now.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From deuce@VERT to CVS commit on Monday, September 28, 2015 17:48:52
    src/xpdev xpprintf.c 1.56 1.57
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv17502

    Modified Files:
    xpprintf.c
    Log Message:
    We need to #define _GNU_SOURCE on Linux before #include<>ing stdio.h.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ telnet://vert.synchro.net
  • From rswindell@VERT to CVS commit on Monday, February 19, 2018 20:50:29
    src/xpdev xpprintf.c 1.57 1.58
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv11910

    Modified Files:
    xpprintf.c
    Log Message:
    Address a RESOURCE LEAK reported by Coverity - unlikely to occur normally.


    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Thursday, March 08, 2018 15:33:13
    src/xpdev xpprintf.c 1.58 1.59
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/home/rswindell/sbbs/src/xpdev

    Modified Files:
    xpprintf.c
    Log Message:
    Don't redefine _GNU_SOURCE if it's already defined.



    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From deuce@VERT to CVS commit on Friday, March 09, 2018 22:42:41
    src/xpdev xpprintf.c 1.60 1.61
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/tmp/cvs-serv20861

    Modified Files:
    xpprintf.c
    Log Message:
    Call va_end() on errors as well.




    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From rswindell@VERT to CVS commit on Saturday, April 07, 2018 00:21:46
    src/xpdev xpprintf.c 1.61 1.62
    Update of /cvsroot/sbbs/src/xpdev
    In directory cvs:/home/rswindell/sbbs/src/xpdev

    Modified Files:
    xpprintf.c
    Log Message:
    Resolve GCC warnings:
    cast from pointer to integer of different size



    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¨@VERT to Git commit to main/sbbs/master on Thursday, February 08, 2024 08:42:00
    https://gitlab.synchro.net/main/sbbs/-/commit/72305c4cfe7afe3106442269
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Fix memory leak on realloc failure in xp_aprintf_next()

    The caller of this function must clobber the passed in pointer,
    and we're returning NULL meaning we would be overwriting the pointer
    to the allocation.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¨@VERT to Git commit to main/sbbs/master on Thursday, February 15, 2024 15:36:21
    https://gitlab.synchro.net/main/sbbs/-/commit/8dff1f0f5bbc9f59a53716fa
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Re-enabled use of strtold() for long doubles... it is likely
    ubiquitous now (except for Borland of course, which doesn't count)

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¨@VERT to Git commit to main/sbbs/master on Friday, February 16, 2024 01:17:08
    https://gitlab.synchro.net/main/sbbs/-/commit/2dd514d0f928f4ab62d50a94
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Use strtoull() converting a string to a size_t

    Except on Borland, which presumably doesn't have strtoull().

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¨@VERT to Git commit to main/sbbs/master on Thursday, February 22, 2024 06:50:40
    https://gitlab.synchro.net/main/sbbs/-/commit/3aab34e03cebd6f0d26fe740
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Remove do-nothings that Coverity complains do nothing.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¨@VERT to Git commit to main/sbbs/master on Thursday, February 22, 2024 06:50:40
    https://gitlab.synchro.net/main/sbbs/-/commit/78e67f2c4d1389c40a184f28
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Avoid pasing NULL to strto*() functions.

    Whill this is perfectly legal and is not a problem, Coverity complains.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Friday, January 10, 2025 18:20:48
    https://gitlab.synchro.net/main/sbbs/-/commit/e0aac8d080458a38b7b30001
    Modified Files:
    src/xpdev/xpprintf.c
    Log Message:
    Resolve MSVC warnings about possible data loss and signed/unsigned compares

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