Fix edge cases in snprintf

Author: Sympatron

src/snprintf.c

@@ -210,6 +210,7 @@ int vsnprintf(
             {
                char s[MAXIMUM_NUMBER_LENGTH] = { 0 };
                signed long long ll = 0;
+               unsigned long long ull = 0;
                if ( is_long == 2 )
                {
                   // Render %lld
@@ -226,11 +227,8 @@ int vsnprintf(
                   ll = va_arg( ap, signed int );
                }
                bool is_negative = ll < 0;
-               if (is_negative)
-               {
-                  ll = -ll;
-               }
-               itoa( ll, s, sizeof(s), 10 );
+               ull = is_negative ? -ll : ll;
+               utoa( ull, s, sizeof(s), 10 );
                write_padding( str, size, &written, strlen(s), width, precision, zero_pad, is_negative );
                for ( const char* p = s; *p != '\0'; p++ )
                {
@@ -430,13 +428,16 @@ static void write_padding(char* restrict str, size_t size, size_t* written, size
    unsigned long zero_pad_len = 0;
    if ( precision != 0 && precision != (unsigned long)-1 )
    {
-      zero_pad_len = precision > len ? precision - len : 0;
       if ( is_negative )
       {
-         zero_pad_len++;
+         zero_pad_len = precision >= len ? precision - len + 1 : 0;
+      }
+      else
+      {
+         zero_pad_len = precision >= len ? precision - len : 0;
       }
    }
-   else if ( zero_pad )
+   else if ( zero_pad && precision == (unsigned long)-1 )
    {
       zero_pad_len = pad_len;
    }

src/snprintf.rs

@@ -94,6 +94,34 @@ mod test {
 		);
 	}
 
+	#[test]
+	fn int_min() {
+		asprintf(
+			"%d",
+			&format!("{}", CInt::min_value()),
+			|buf, len, fmt| unsafe { snprintf(buf, len, fmt, CInt::min_value()) },
+		);
+		asprintf(
+			"%lld",
+			&format!("{}", CLongLong::min_value()),
+			|buf, len, fmt| unsafe { snprintf(buf, len, fmt, CLongLong::min_value()) },
+		);
+	}
+
+	#[test]
+	fn int_max() {
+		asprintf(
+			"%d",
+			&format!("{}", CInt::max_value()),
+			|buf, len, fmt| unsafe { snprintf(buf, len, fmt, CInt::max_value()) },
+		);
+		asprintf(
+			"%lld",
+			&format!("{}", CLongLong::max_value()),
+			|buf, len, fmt| unsafe { snprintf(buf, len, fmt, CLongLong::max_value()) },
+		);
+	}
+
 	#[test]
 	fn non_null_terminated_with_length() {
 		asprintf("%.*s", "01234", |buf, len, fmt: *const u8| unsafe {
@@ -246,5 +274,15 @@ mod test {
 		asprintf("%5.0d", " -123", |buf, len, fmt| unsafe {
 			snprintf(buf, len, fmt, CInt::from(-123i8))
 		});
+
+		asprintf("%05.4d", "-0123", |buf, len, fmt| unsafe {
+			snprintf(buf, len, fmt, CInt::from(-123i8))
+		});
+		asprintf("%05.3d", " -123", |buf, len, fmt| unsafe {
+			snprintf(buf, len, fmt, CInt::from(-123i8))
+		});
+		asprintf("%05.0d", " -123", |buf, len, fmt| unsafe {
+			snprintf(buf, len, fmt, CInt::from(-123i8))
+		});
 	}
 }