Return values and error handling for guac_send_* functions in protocol.

This commit is contained in:
Michael Jumper 2011-03-19 16:32:35 -07:00
parent 5a2a629225
commit 6300c7c56a
2 changed files with 132 additions and 91 deletions

View File

@ -132,8 +132,9 @@ char* guac_unescape_string_inplace(char* str);
* *
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param args The NULL-terminated array of argument names (strings). * @param args The NULL-terminated array of argument names (strings).
* @return Zero on success, non-zero on error.
*/ */
void guac_send_args(GUACIO* io, const char** name); int guac_send_args(GUACIO* io, const char** name);
/** /**
* Sends a name instruction over the given GUACIO connection. The * Sends a name instruction over the given GUACIO connection. The
@ -141,8 +142,9 @@ void guac_send_args(GUACIO* io, const char** name);
* *
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param name The name to send within the name instruction. * @param name The name to send within the name instruction.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_name(GUACIO* io, const char* name); int guac_send_name(GUACIO* io, const char* name);
/** /**
* Sends a sync instruction over the given GUACIO connection. The * Sends a sync instruction over the given GUACIO connection. The
@ -150,8 +152,9 @@ void guac_send_name(GUACIO* io, const char* name);
* *
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param timestamp The current timestamp (in milliseconds). * @param timestamp The current timestamp (in milliseconds).
* @return Zero on success, non-zero on error.
*/ */
void guac_send_sync(GUACIO* io, long timestamp); int guac_send_sync(GUACIO* io, long timestamp);
/** /**
* Sends an error instruction over the given GUACIO connection. The * Sends an error instruction over the given GUACIO connection. The
@ -160,8 +163,9 @@ void guac_send_sync(GUACIO* io, long timestamp);
* *
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param error The description associated with the error. * @param error The description associated with the error.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_error(GUACIO* io, const char* error); int guac_send_error(GUACIO* io, const char* error);
/** /**
* Sends a clipboard instruction over the given GUACIO connection. The * Sends a clipboard instruction over the given GUACIO connection. The
@ -169,8 +173,9 @@ void guac_send_error(GUACIO* io, const char* error);
* *
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param data The clipboard data to send. * @param data The clipboard data to send.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_clipboard(GUACIO* io, const char* data); int guac_send_clipboard(GUACIO* io, const char* data);
/** /**
* Sends a size instruction over the given GUACIO connection. * Sends a size instruction over the given GUACIO connection.
@ -178,8 +183,9 @@ void guac_send_clipboard(GUACIO* io, const char* data);
* @param io The GUACIO connection to use. * @param io The GUACIO connection to use.
* @param w The width of the display. * @param w The width of the display.
* @param h The height of the display. * @param h The height of the display.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_size(GUACIO* io, int w, int h); int guac_send_size(GUACIO* io, int w, int h);
/** /**
* Sends a copy instruction over the given GUACIO connection. * Sends a copy instruction over the given GUACIO connection.
@ -195,8 +201,9 @@ void guac_send_size(GUACIO* io, int w, int h);
* should be copied. * should be copied.
* @param dsty The Y coordinate of the destination, where the source rectangle * @param dsty The Y coordinate of the destination, where the source rectangle
* should be copied. * should be copied.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_copy(GUACIO* io, int srcl, int srcx, int srcy, int w, int h, int guac_send_copy(GUACIO* io, int srcl, int srcx, int srcy, int w, int h,
int dstl, int dstx, int dsty); int dstl, int dstx, int dsty);
/** /**
@ -211,8 +218,9 @@ void guac_send_copy(GUACIO* io, int srcl, int srcx, int srcy, int w, int h,
* data to send. * data to send.
* @param w The width of the image in the image buffer. * @param w The width of the image in the image buffer.
* @param h The height of the image in the image buffer. * @param h The height of the image in the image buffer.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_png(GUACIO* io, int layer, int x, int y, int guac_send_png(GUACIO* io, int layer, int x, int y,
png_byte** png_rows, int w, int h); png_byte** png_rows, int w, int h);
/** /**
@ -226,8 +234,9 @@ void guac_send_png(GUACIO* io, int layer, int x, int y,
* data to send. * data to send.
* @param w The width of the image in the image buffer. * @param w The width of the image in the image buffer.
* @param h The height of the image in the image buffer. * @param h The height of the image in the image buffer.
* @return Zero on success, non-zero on error.
*/ */
void guac_send_cursor(GUACIO* io, int x, int y, int guac_send_cursor(GUACIO* io, int x, int y,
png_byte** png_rows, int w, int h); png_byte** png_rows, int w, int h);
/** /**

View File

@ -167,97 +167,127 @@ char* guac_unescape_string_inplace(char* str) {
} }
void guac_send_args(GUACIO* io, const char** args) { int guac_send_args(GUACIO* io, const char** args) {
int i; int i;
guac_write_string(io, "args:"); if (guac_write_string(io, "args:")) return -1;
for (i=0; args[i] != NULL; i++) { for (i=0; args[i] != NULL; i++) {
if (i > 0) if (i > 0) {
guac_write_string(io, ","); if (guac_write_string(io, ","))
return -1;
}
char* escaped = guac_escape_string(args[i]); char* escaped = guac_escape_string(args[i]);
guac_write_string(io, escaped); if (guac_write_string(io, escaped)) {
free(escaped);
return -1;
}
free(escaped); free(escaped);
} }
guac_write_string(io, ";"); return guac_write_string(io, ";");
} }
void guac_send_name(GUACIO* io, const char* name) { int guac_send_name(GUACIO* io, const char* name) {
char* escaped = guac_escape_string(name); char* escaped = guac_escape_string(name);
guac_write_string(io, "name:"); if (
guac_write_string(io, escaped); guac_write_string(io, "name:")
guac_write_string(io, ";"); || guac_write_string(io, escaped)
|| guac_write_string(io, ";")
) {
free(escaped);
return -1;
}
free(escaped); free(escaped);
return 0;
} }
void guac_send_size(GUACIO* io, int w, int h) { int guac_send_size(GUACIO* io, int w, int h) {
guac_write_string(io, "size:");
guac_write_int(io, w); return
guac_write_string(io, ","); guac_write_string(io, "size:")
guac_write_int(io, h); || guac_write_int(io, w)
guac_write_string(io, ";"); || guac_write_string(io, ",")
|| guac_write_int(io, h)
|| guac_write_string(io, ";");
} }
void guac_send_clipboard(GUACIO* io, const char* data) { int guac_send_clipboard(GUACIO* io, const char* data) {
char* escaped = guac_escape_string(data); char* escaped = guac_escape_string(data);
guac_write_string(io, "clipboard:"); if (
guac_write_string(io, escaped); guac_write_string(io, "clipboard:")
guac_write_string(io, ";"); || guac_write_string(io, escaped)
|| guac_write_string(io, ";")
) {
free(escaped);
return -1;
}
free(escaped); free(escaped);
return 0;
} }
void guac_send_error(GUACIO* io, const char* error) { int guac_send_error(GUACIO* io, const char* error) {
char* escaped = guac_escape_string(error); char* escaped = guac_escape_string(error);
guac_write_string(io, "error:"); if (
guac_write_string(io, escaped); guac_write_string(io, "error:")
guac_write_string(io, ";"); || guac_write_string(io, escaped)
|| guac_write_string(io, ";")
) {
free(escaped);
return -1;
}
free(escaped); free(escaped);
return 0;
} }
void guac_send_sync(GUACIO* io, long timestamp) { int guac_send_sync(GUACIO* io, long timestamp) {
guac_write_string(io, "sync:"); return
guac_write_int(io, timestamp); guac_write_string(io, "sync:")
guac_write_string(io, ";"); || guac_write_int(io, timestamp)
|| guac_write_string(io, ";");
} }
void guac_send_copy(GUACIO* io, int srcl, int srcx, int srcy, int w, int h, int dstl, int dstx, int dsty) { int guac_send_copy(GUACIO* io, int srcl, int srcx, int srcy, int w, int h, int dstl, int dstx, int dsty) {
guac_write_string(io, "copy:");
guac_write_int(io, srcl); return
guac_write_string(io, ","); guac_write_string(io, "copy:")
guac_write_int(io, srcx); || guac_write_int(io, srcl)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, srcy); || guac_write_int(io, srcx)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, w); || guac_write_int(io, srcy)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, h); || guac_write_int(io, w)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, dstl); || guac_write_int(io, h)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, dstx); || guac_write_int(io, dstl)
guac_write_string(io, ","); || guac_write_string(io, ",")
guac_write_int(io, dsty); || guac_write_int(io, dstx)
guac_write_string(io, ";"); || guac_write_string(io, ",")
|| guac_write_int(io, dsty)
|| guac_write_string(io, ";");
} }
void __guac_write_png(png_structp png, png_bytep data, png_size_t length) { void __guac_write_png(png_structp png, png_bytep data, png_size_t length) {
@ -281,7 +311,7 @@ void __guac_write_png(png_structp png, png_bytep data, png_size_t length) {
void __guac_write_flush(png_structp png) { void __guac_write_flush(png_structp png) {
} }
void guac_send_png(GUACIO* io, int layer, int x, int y, png_byte** png_rows, int w, int h) { int guac_send_png(GUACIO* io, int layer, int x, int y, png_byte** png_rows, int w, int h) {
png_structp png; png_structp png;
png_infop png_info; png_infop png_info;
@ -291,22 +321,19 @@ void guac_send_png(GUACIO* io, int layer, int x, int y, png_byte** png_rows, int
/* Set up PNG writer */ /* Set up PNG writer */
png = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); png = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
if (!png) { if (!png) {
perror("Error initializing libpng write structure"); return -1;
return;
} }
png_info = png_create_info_struct(png); png_info = png_create_info_struct(png);
if (!png_info) { if (!png_info) {
perror("Error initializing libpng info structure");
png_destroy_write_struct(&png, NULL); png_destroy_write_struct(&png, NULL);
return; return -1;
} }
/* Set error handler */ /* Set error handler */
if (setjmp(png_jmpbuf(png))) { if (setjmp(png_jmpbuf(png))) {
perror("Error setting handler");
png_destroy_write_struct(&png, &png_info); png_destroy_write_struct(&png, &png_info);
return; return -1;
} }
png_set_write_fn(png, io, __guac_write_png, __guac_write_flush); png_set_write_fn(png, io, __guac_write_png, __guac_write_flush);
@ -324,30 +351,34 @@ void guac_send_png(GUACIO* io, int layer, int x, int y, png_byte** png_rows, int
PNG_FILTER_TYPE_DEFAULT PNG_FILTER_TYPE_DEFAULT
); );
guac_write_string(io, "png:"); if (
guac_write_int(io, layer); guac_write_string(io, "png:")
guac_write_string(io, ","); || guac_write_int(io, layer)
guac_write_int(io, x); || guac_write_string(io, ",")
guac_write_string(io, ","); || guac_write_int(io, x)
guac_write_int(io, y); || guac_write_string(io, ",")
guac_write_string(io, ","); || guac_write_int(io, y)
|| guac_write_string(io, ",")
) {
png_destroy_write_struct(&png, &png_info);
return -1;
}
png_set_rows(png, png_info, png_rows); png_set_rows(png, png_info, png_rows);
png_write_png(png, png_info, PNG_TRANSFORM_IDENTITY, NULL); png_write_png(png, png_info, PNG_TRANSFORM_IDENTITY, NULL);
if (guac_flush_base64(io) < 0) { if (guac_flush_base64(io) < 0) {
perror("Error flushing PNG"); png_destroy_write_struct(&png, &png_info);
png_error(png, "Error flushing PNG"); return -1;
return;
} }
png_destroy_write_struct(&png, &png_info); png_destroy_write_struct(&png, &png_info);
return guac_write_string(io, ";");
guac_write_string(io, ";");
} }
void guac_send_cursor(GUACIO* io, int x, int y, png_byte** png_rows, int w, int h) { int guac_send_cursor(GUACIO* io, int x, int y, png_byte** png_rows, int w, int h) {
png_structp png; png_structp png;
png_infop png_info; png_infop png_info;
@ -357,22 +388,19 @@ void guac_send_cursor(GUACIO* io, int x, int y, png_byte** png_rows, int w, int
/* Set up PNG writer */ /* Set up PNG writer */
png = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); png = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
if (!png) { if (!png) {
perror("Error initializing libpng write structure"); return -1;
return;
} }
png_info = png_create_info_struct(png); png_info = png_create_info_struct(png);
if (!png_info) { if (!png_info) {
perror("Error initializing libpng info structure");
png_destroy_write_struct(&png, NULL); png_destroy_write_struct(&png, NULL);
return; return -1;
} }
/* Set error handler */ /* Set error handler */
if (setjmp(png_jmpbuf(png))) { if (setjmp(png_jmpbuf(png))) {
perror("Error setting handler");
png_destroy_write_struct(&png, &png_info); png_destroy_write_struct(&png, &png_info);
return; return -1;
} }
png_set_write_fn(png, io, __guac_write_png, __guac_write_flush); png_set_write_fn(png, io, __guac_write_png, __guac_write_flush);
@ -390,23 +418,27 @@ void guac_send_cursor(GUACIO* io, int x, int y, png_byte** png_rows, int w, int
PNG_FILTER_TYPE_DEFAULT PNG_FILTER_TYPE_DEFAULT
); );
guac_write_string(io, "cursor:"); if (
guac_write_int(io, x); guac_write_string(io, "cursor:")
guac_write_string(io, ","); || guac_write_int(io, x)
guac_write_int(io, y); || guac_write_string(io, ",")
guac_write_string(io, ","); || guac_write_int(io, y)
|| guac_write_string(io, ",")
) {
png_destroy_write_struct(&png, &png_info);
return -1;
}
png_set_rows(png, png_info, png_rows); png_set_rows(png, png_info, png_rows);
png_write_png(png, png_info, PNG_TRANSFORM_IDENTITY, NULL); png_write_png(png, png_info, PNG_TRANSFORM_IDENTITY, NULL);
if (guac_flush_base64(io) < 0) { if (guac_flush_base64(io) < 0) {
perror("Error flushing PNG"); png_destroy_write_struct(&png, &png_info);
png_error(png, "Error flushing PNG"); return -1;
return;
} }
png_destroy_write_struct(&png, &png_info); png_destroy_write_struct(&png, &png_info);
return guac_write_string(io, ";");
guac_write_string(io, ";");
} }