[coreboot] Patch set updated for coreboot: 870e1f8 cbfstool: Add buffer management API.

Hung-Te Lin (hungte@chromium.org) gerrit at coreboot.org
Wed Jan 30 05:05:16 CET 2013


Hung-Te Lin (hungte at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2205

-gerrit

commit 870e1f81c3cb1fde99d35c8cac1de5b4fcaf38e9
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Wed Jan 30 00:43:46 2013 +0800

    cbfstool: Add buffer management API.
    
    Many functions in cbfstool need to deal with a memory buffer - both location and
    size. Right now it's made by different ways: for ROM image using global variable
    (romsize, master_header); and in cbfs-* using return value for size and char**
    to return memory location.
    
    This may cause bugs like assuming incorrect return types, ex:
    	uint32_t file_size = parse();	// which returns "-1" on error
    	if (file_size <= 0) { ...
    And the parse error will never be caught.
    
    We can simplify this by introducing a buffer API, to change
    	unsigned int do_something(char *input, size_t len, char **output, ...)
    into
    	int do_something(struct buffer *input, struct buffer *output, ...)
    
    The buffer API will be used by further commits.
    
    Change-Id: Iaddaeb109f08be6be84c6728d72c6a043b0e7a9f
    Signed-off-by: Hung-Te Lin <hungte at chromium.org>
---
 util/cbfstool/common.c | 63 ++++++++++++++++++++++++++++++++++++++++++++++++++
 util/cbfstool/common.h | 23 ++++++++++++++++++
 2 files changed, 86 insertions(+)

diff --git a/util/cbfstool/common.c b/util/cbfstool/common.c
index 97bf168..fd82b6c 100644
--- a/util/cbfstool/common.c
+++ b/util/cbfstool/common.c
@@ -29,6 +29,69 @@
 
 #define dprintf(x...)
 
+/* Buffer and file I/O */
+
+int buffer_create(struct buffer *buffer, size_t size, const char *name) {
+	buffer->name = strdup(name);
+	buffer->size = size;
+	buffer->data = (char *)malloc(buffer->size);
+	if (!buffer->data) {
+		fprintf(stderr, "buffer_create: Insufficient memory (0x%zx).\n",
+			size);
+	}
+	return (buffer->data == NULL);
+}
+
+int buffer_from_file(struct buffer *buffer, const char *filename) {
+	FILE *fp = fopen(filename, "rb");
+	if (!fp) {
+		perror(filename);
+		return -1;
+	}
+	fseek(fp, 0, SEEK_END);
+	buffer->size = ftell(fp);
+	buffer->name = strdup(filename);
+	rewind(fp);
+	buffer->data = (char *)malloc(buffer->size);
+	assert(buffer->data);
+	if (fread(buffer->data, 1, buffer->size, fp) != buffer->size) {
+		fprintf(stderr, "incomplete read: %s\n", filename);
+		fclose(fp);
+		return -1;
+	}
+	fclose(fp);
+	return 0;
+}
+
+int buffer_write_file(struct buffer *buffer, const char *filename) {
+	FILE *fp = fopen(filename, "wb");
+	if (!fp) {
+		perror(filename);
+		return -1;
+	}
+	assert(buffer && buffer->data);
+	if (fwrite(buffer->data, 1, buffer->size, fp) != buffer->size) {
+		fprintf(stderr, "incomplete write: %s\n", filename);
+		fclose(fp);
+		return -1;
+	}
+	fclose(fp);
+	return 0;
+}
+
+void buffer_delete(struct buffer *buffer) {
+	assert(buffer);
+	if (buffer->name) {
+		free(buffer->name);
+		buffer->name = NULL;
+	}
+	if (buffer->data) {
+		free(buffer->data);
+		buffer->data = NULL;
+	}
+	buffer->size = 0;
+}
+
 size_t getfilesize(const char *filename)
 {
 	size_t size;
diff --git a/util/cbfstool/common.h b/util/cbfstool/common.h
index 853e529..77bbb5c 100644
--- a/util/cbfstool/common.h
+++ b/util/cbfstool/common.h
@@ -21,6 +21,8 @@
 #define __CBFSTOOL_COMMON_H
 
 #include <stdint.h>
+#include <assert.h>
+
 #include "swab.h"
 #ifndef __APPLE__
 #define ntohl(x)  (host_bigendian?(x):swab32(x))
@@ -29,6 +31,27 @@
 #define ntohll(x) (host_bigendian?(x):swab64(x))
 #define htonll(x) (host_bigendian?(x):swab64(x))
 
+/* Buffer and file I/O */
+struct buffer {
+	char *name;
+	char *data;
+	size_t size;
+};
+
+/* Creates an empty memory buffer with given size.
+ * Returns 0 on success, otherwise non-zero. */
+int buffer_create(struct buffer *buffer, size_t size, const char *name);
+
+/* Loads a file into memory buffer. Returns 0 on success, otherwise non-zero. */
+int buffer_from_file(struct buffer *buffer, const char *filename);
+
+/* Writes memory buffer content into file.
+ * Returns 0 on success, otherwise non-zero. */
+int buffer_write_file(struct buffer *buffer, const char *filename);
+
+/* Destroys a memory buffer. */
+void buffer_delete(struct buffer *buffer);
+
 extern void *offset;
 extern uint32_t romsize;
 extern int host_bigendian;



More information about the coreboot mailing list