vis

commit 7db58f02140d34f73f3bce943010395497642d53
parent bbf096d84d319ae24ec31716fbf7ae5e776f751a
Author: Marc André Tanner <mat@brain-dump.org>
Date:   Sun, 10 Jan 2016 12:23:32 +0100

text: introduce text_bytes_alloc0 utility function

Heap allocates a zero terminated string of the given range.
Freeing is the caller's responsibility.

Checks for unsigned integer overflow i.e. passing SIZE_MAX
as len will always fail because there is no room for the
terminating NUL byte. This is important as EPOS is defined
to be SIZE_MAX.

Diffstat:
M
text.c
 | 
12
++++++++++++
M
text.h
 | 
3
+++

2 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/text.c b/text.c
@@ -21,6 +21,7 @@
 #include <fcntl.h>
 #include <errno.h>
 #include <wchar.h>
+#include <stdint.h>
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <sys/mman.h>
@@ -1441,6 +1442,17 @@ size_t text_bytes_get(Text *txt, size_t pos, size_t len, char *buf) {
 	return len - rem;
 }
 
+char *text_bytes_alloc0(Text *txt, size_t pos, size_t len) {
+	if (len == SIZE_MAX)
+		return NULL;
+	char *buf = malloc(len+1);
+	if (!buf)
+		return NULL;
+	len = text_bytes_get(txt, pos, len, buf);
+	buf[len] = '\0';
+	return buf;
+}
+
 size_t text_size(Text *txt) {
 	return txt->size;
 }
diff --git a/text.h b/text.h
@@ -72,6 +72,9 @@ bool text_byte_get(Text*, size_t pos, char *buf);
  * indicates how many bytes were copied into `buf'. WARNING buf will not be
  * NUL terminated. */
 size_t text_bytes_get(Text*, size_t pos, size_t len, char *buf);
+/* allocate a NUL terminated buffer and fill at most `len' bytes
+ * starting at `pos'. Freeing is the caller's responsibility! */
+char *text_bytes_alloc0(Text*, size_t pos, size_t len);
 
 Iterator text_iterator_get(Text*, size_t pos);
 bool text_iterator_valid(const Iterator*);