diff options
author | YAMAMOTO Takashi <yamamoto@midokura.com> | 2016-03-16 07:19:35 +0000 |
---|---|---|
committer | YAMAMOTO Takashi <yamamoto@midokura.com> | 2016-03-24 01:09:53 +0900 |
commit | 11b7f938280797676d2163c765916c8c904251db (patch) | |
tree | da0a9656d425945def092d31e8e3988cd2174bd3 /lib | |
parent | 1de0ef4147f52c5d715e380d1f394c7e1302e38f (diff) | |
download | openvswitch-11b7f938280797676d2163c765916c8c904251db.tar.gz |
ovs_strerror, ovs_format_message: Always use "Success" for errno 0
So that testsuite can compare log messages including the string.
Signed-off-by: YAMAMOTO Takashi <yamamoto@midokura.com>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/util.c | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/lib/util.c b/lib/util.c index e3519f2bd..bdc78f5fb 100644 --- a/lib/util.c +++ b/lib/util.c @@ -413,6 +413,21 @@ ovs_strerror(int error) char *buffer; char *s; + if (error == 0) { + /* + * strerror(0) varies among platforms: + * + * Success + * No error + * Undefined error: 0 + * + * We want to provide a consistent result here because + * our testsuite has test cases which strictly matches + * log messages containing this string. + */ + return "Success"; + } + save_errno = errno; buffer = strerror_buffer_get()->s; @@ -2060,6 +2075,11 @@ ovs_format_message(int error) enum { BUFSIZE = sizeof strerror_buffer_get()->s }; char *buffer = strerror_buffer_get()->s; + if (error == 0) { + /* See ovs_strerror */ + return "Success"; + } + FormatMessage(FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_IGNORE_INSERTS, NULL, error, 0, buffer, BUFSIZE, NULL); return buffer; |