From: Ben Pfaff Date: Thu, 31 Mar 2011 21:50:20 +0000 (-0700) Subject: Add a few more users for ovs_retval_to_string(). X-Git-Tag: v1.2.0~490 X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=fe1e967e3bc6c04f773b150fbb358b2350d0ad8c;p=sliver-openvswitch.git Add a few more users for ovs_retval_to_string(). --- diff --git a/lib/entropy.c b/lib/entropy.c index 8a6479bef..1f1af50a5 100644 --- a/lib/entropy.c +++ b/lib/entropy.c @@ -47,8 +47,7 @@ get_entropy(void *buffer, size_t n) close(fd); if (error) { - VLOG_ERR("%s: read error (%s)", urandom, - error == EOF ? "unexpected end of file" : strerror(error)); + VLOG_ERR("%s: read error (%s)", urandom, ovs_retval_to_string(error)); } return error; } diff --git a/lib/ovsdb-error.c b/lib/ovsdb-error.c index 3b90b1616..0ac93dc65 100644 --- a/lib/ovsdb-error.c +++ b/lib/ovsdb-error.c @@ -192,12 +192,6 @@ ovsdb_error_clone(const struct ovsdb_error *old) } } -static const char * -ovsdb_errno_string(int error) -{ - return error == EOF ? "unexpected end of file" : strerror(error); -} - struct json * ovsdb_error_to_json(const struct ovsdb_error *error) { @@ -211,7 +205,7 @@ ovsdb_error_to_json(const struct ovsdb_error *error) } if (error->errno_) { json_object_put_string(json, "io-error", - ovsdb_errno_string(error->errno_)); + ovs_retval_to_string(error->errno_)); } return json; } @@ -228,7 +222,7 @@ ovsdb_error_to_string(const struct ovsdb_error *error) ds_put_format(&ds, ": %s", error->details); } if (error->errno_) { - ds_put_format(&ds, " (%s)", ovsdb_errno_string(error->errno_)); + ds_put_format(&ds, " (%s)", ovs_retval_to_string(error->errno_)); } return ds_steal_cstr(&ds); } diff --git a/lib/unixctl.c b/lib/unixctl.c index 6e0b19ff6..c333c54e9 100644 --- a/lib/unixctl.c +++ b/lib/unixctl.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008, 2009, 2010 Nicira Networks. + * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -568,8 +568,7 @@ unixctl_client_transact(struct unixctl_client *client, if (error) { VLOG_WARN("error reading reply from %s: %s", client->connect_path, - (error == EOF ? "unexpected end of file" - : strerror(error))); + ovs_retval_to_string(error)); goto error; } diff --git a/tests/ovsdb-log.at b/tests/ovsdb-log.at index 507ef8ea9..f7b6432e3 100644 --- a/tests/ovsdb-log.at +++ b/tests/ovsdb-log.at @@ -235,7 +235,7 @@ AT_CHECK( [[file: open successful file: read: [0] file: read: [1] -file: read failed: I/O error: file: error reading 4 bytes starting at offset 170 (unexpected end of file) +file: read failed: I/O error: file: error reading 4 bytes starting at offset 170 (End of file) file: write:["longer data"] successful ]], [ignore]) AT_CHECK(