• src/xpdev/xpprintf.c

    From deuce@VERT to CVS commit on Sat Oct 12 23:12:22 2013
    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 þ Vert
  • From deuce@VERT to CVS commit on Wed Feb 5 02:22:17 2014
    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 þ Vertr
  • From deuce@VERT to CVS commit on Wed Feb 5 19:13:53 2014
    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://
  • From deuce@VERT to CVS commit on Wed Apr 23 22:46:55 2014
    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 Thu Apr 24 00:02:59 2014
    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 Thu Apr 24 00:06:00 2014
    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 Thu Apr 24 00:52:51 2014
    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 Thu Apr 24 00:58:06 2014
    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 Tue Apr 29 02:24:39 2014
    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 Mon Oct 6 14:45:36 2014
    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 Sun Sep 27 23:05:20 2015
    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 Sun Sep 27 23:09:56 2015
    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 Mon Sep 28 13:26:51 2015
    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 Mon Sep 28 13:31:48 2015
    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 Mon Sep 28 13:34:54 2015
    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 Mon Sep 28 17:44:20 2015
    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 Mon Sep 28 17:47:48 2015
    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 Mon Sep 28 17:48:52 2015
    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 Mon Feb 19 20:50:29 2018
    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 Thu Mar 8 15:33:13 2018
    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 Fri Mar 9 22:42:41 2018
    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 Sat Apr 7 00:21:46 2018
    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 Thu Feb 8 08:42:00 2024
    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 Thu Feb 15 15:36:21 2024
    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 Fri Feb 16 01:17:08 2024
    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 Thu Feb 22 06:50:40 2024
    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 Thu Feb 22 06:50:40 2024
    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 Fri Jan 10 18:20:48 2025
    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