scc

commit 1e2bd8326fecf0902ac5ed20b487a44ef9333e58
parent 96d8af8bab21bdf79a28fc4608c76b42a96c040f
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;
 		}
 	}