scc

commit 934e2128978ff00ff47a6bd148e209f73b4b6252
parent 944846e259100c68c2398bb1ac3b180543f84a5e
Author: Roberto E. Vargas Caballero <k0ga@shike2.com>
Date:   Fri,  8 Oct 2021 12:43:36 +0200

libmach: Remove duplicated va_end() in pack/unpack

These functions were transformed from normal variable
length argument functions to functions that were accepting
a va_list. In case of error there was still va_end() calls,
that would mean that the va could be ended twice.

Diffstat:
M
src/libmach/pack.c
 | 
2
--
M
src/libmach/unpack.c
 | 
2
--

2 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/src/libmach/pack.c b/src/libmach/pack.c
@@ -53,7 +53,6 @@ lpack(unsigned char *dst, char *fmt, va_list va)
 			*bp++ = q >> 56;
 			break;
 		default:
-			va_end(va);
 			return -1;
 		}
 	}
@@ -108,7 +107,6 @@ bpack(unsigned char *dst, char *fmt, va_list va)
 			*bp++ = q;
 			break;
 		default:
-			va_end(va);
 			return -1;
 		}
 	}
diff --git a/src/libmach/unpack.c b/src/libmach/unpack.c
@@ -58,7 +58,6 @@ lunpack(unsigned char *src, char *fmt, va_list va)
 			*qp = q;
 			break;
 		default:
-			va_end(va);
 			return -1;
 		}
 	}
@@ -118,7 +117,6 @@ bunpack(unsigned char *src, char *fmt, va_list va)
 			*qp = q;
 			break;
 		default:
-			va_end(va);
 			return -1;
 		}
 	}