summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLars-Dominik Braun <lars@6xq.net>2011-09-19 20:52:45 +0200
committerLars-Dominik Braun <lars@6xq.net>2011-11-09 20:10:03 +0100
commite2e80f0890ec52ad7856850761bff0840bbb65b4 (patch)
tree9c96bc61399828e48157e12c474dd1bb740ed60b
parente0b076eb06f605cae91022d2e644cda2e5ea5676 (diff)
downloadpianobar-windows-e2e80f0890ec52ad7856850761bff0840bbb65b4.tar.gz
pianobar-windows-e2e80f0890ec52ad7856850761bff0840bbb65b4.tar.bz2
pianobar-windows-e2e80f0890ec52ad7856850761bff0840bbb65b4.zip
waitress: handler return values
-rw-r--r--src/libwaitress/waitress.c56
-rw-r--r--src/libwaitress/waitress.h10
2 files changed, 50 insertions, 16 deletions
diff --git a/src/libwaitress/waitress.c b/src/libwaitress/waitress.c
index a0b6932..1c2456f 100644
--- a/src/libwaitress/waitress.c
+++ b/src/libwaitress/waitress.c
@@ -513,16 +513,20 @@ static char *WaitressGetline (char * const str) {
/* identity encoding handler
*/
-static WaitressCbReturn_t WaitressHandleIdentity (WaitressHandle_t *waith,
+static WaitressHandlerReturn_t WaitressHandleIdentity (WaitressHandle_t *waith,
char *buf, size_t size) {
waith->request.contentReceived += size;
- return waith->callback (buf, size, waith->data);
+ if (waith->callback (buf, size, waith->data) == WAITRESS_CB_RET_ERR) {
+ return WAITRESS_HANDLER_ABORTED;
+ } else {
+ return WAITRESS_HANDLER_CONTINUE;
+ }
}
/* chunked encoding handler. buf must be \0-terminated, size does not include
* trailing \0.
*/
-static WaitressCbReturn_t WaitressHandleChunked (WaitressHandle_t *waith,
+static WaitressHandlerReturn_t WaitressHandleChunked (WaitressHandle_t *waith,
char *buf, size_t size) {
char *content = buf, *nextContent;
@@ -538,26 +542,26 @@ static WaitressCbReturn_t WaitressHandleChunked (WaitressHandle_t *waith,
} else {
WaitressHandleIdentity (waith, content, remaining);
waith->request.chunkSize -= remaining;
- return WAITRESS_CB_RET_OK;
+ return WAITRESS_HANDLER_CONTINUE;
}
}
if ((nextContent = WaitressGetline (content)) != NULL) {
long int chunkSize = strtol (content, NULL, 16);
if (chunkSize == 0) {
- return WAITRESS_CB_RET_OK;
+ return WAITRESS_HANDLER_DONE;
} else if (chunkSize < 0) {
- return WAITRESS_CB_RET_ERR;
+ return WAITRESS_HANDLER_ERR;
} else {
waith->request.chunkSize = chunkSize;
content = nextContent;
}
} else {
- return WAITRESS_CB_RET_OK;
+ return WAITRESS_HANDLER_CONTINUE;
}
}
- return WAITRESS_CB_RET_OK;
+ return WAITRESS_HANDLER_CONTINUE;
}
/* handle http header
@@ -797,9 +801,18 @@ WaitressReturn_t WaitressFetchCall (WaitressHandle_t *waith) {
if (bufFilled > 0) {
/* data must be \0-terminated for chunked handler */
buf[bufFilled] = '\0';
- if (waith->request.dataHandler (waith, buf, bufFilled) ==
- WAITRESS_CB_RET_ERR) {
- FINISH (WAITRESS_RET_CB_ABORT);
+ switch (waith->request.dataHandler (waith, buf, bufFilled)) {
+ case WAITRESS_HANDLER_DONE:
+ FINISH (WAITRESS_RET_OK);
+ break;
+
+ case WAITRESS_HANDLER_ERR:
+ FINISH (WAITRESS_RET_DECODING_ERR);
+ break;
+
+ case WAITRESS_HANDLER_ABORTED:
+ FINISH (WAITRESS_RET_CB_ABORT);
+ break;
}
}
@@ -808,10 +821,19 @@ WaitressReturn_t WaitressFetchCall (WaitressHandle_t *waith) {
READ_RET (buf, WAITRESS_BUFFER_SIZE-1, &recvSize);
buf[recvSize] = '\0';
if (recvSize > 0) {
- if (waith->request.dataHandler (waith, buf, recvSize) ==
- WAITRESS_CB_RET_ERR) {
- wRet = WAITRESS_RET_CB_ABORT;
- break;
+ /* FIXME: Copy&waste */
+ switch (waith->request.dataHandler (waith, buf, recvSize)) {
+ case WAITRESS_HANDLER_DONE:
+ FINISH (WAITRESS_RET_OK);
+ break;
+
+ case WAITRESS_HANDLER_ERR:
+ FINISH (WAITRESS_RET_DECODING_ERR);
+ break;
+
+ case WAITRESS_HANDLER_ABORTED:
+ FINISH (WAITRESS_RET_CB_ABORT);
+ break;
}
}
} while (recvSize > 0);
@@ -885,6 +907,10 @@ const char *WaitressErrorToStr (WaitressReturn_t wRet) {
return "Connection closed by remote host.";
break;
+ case WAITRESS_RET_DECODING_ERR:
+ return "Invalid encoded data.";
+ break;
+
default:
return "No error message available.";
break;
diff --git a/src/libwaitress/waitress.h b/src/libwaitress/waitress.h
index f3aff55..a5aae32 100644
--- a/src/libwaitress/waitress.h
+++ b/src/libwaitress/waitress.h
@@ -39,6 +39,13 @@ typedef enum {
WAITRESS_CB_RET_OK,
} WaitressCbReturn_t;
+typedef enum {
+ WAITRESS_HANDLER_CONTINUE,
+ WAITRESS_HANDLER_DONE,
+ WAITRESS_HANDLER_ERR,
+ WAITRESS_HANDLER_ABORTED,
+} WaitressHandlerReturn_t;
+
typedef struct {
char *url; /* splitted url, unusable */
const char *user;
@@ -64,7 +71,7 @@ typedef struct {
struct {
size_t contentLength, contentReceived, chunkSize;
/* first argument is WaitressHandle_t, but that's not defined here */
- WaitressCbReturn_t (*dataHandler) (void *, char *, size_t);
+ WaitressHandlerReturn_t (*dataHandler) (void *, char *, size_t);
} request;
} WaitressHandle_t;
@@ -82,6 +89,7 @@ typedef enum {
WAITRESS_RET_TIMEOUT,
WAITRESS_RET_READ_ERR,
WAITRESS_RET_CONNECTION_CLOSED,
+ WAITRESS_RET_DECODING_ERR,
} WaitressReturn_t;
void WaitressInit (WaitressHandle_t *);