X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=lib%2Fovsdb-error.c;h=3b90b1616b3545c6b9e17c43b0d63f50f3006afa;hb=77fdfa9bf71aff101219b249955842f6dc59b5c9;hp=c9221493dd8fce1dbb0f47007b0190d97908bf55;hpb=5f55c39b21e69025045437ffbd3bb98fe6ce2e89;p=openvswitch diff --git a/lib/ovsdb-error.c b/lib/ovsdb-error.c index c9221493..3b90b161 100644 --- a/lib/ovsdb-error.c +++ b/lib/ovsdb-error.c @@ -1,4 +1,4 @@ -/* Copyright (c) 2009, 2010 Nicira Networks +/* Copyright (c) 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. @@ -25,8 +25,7 @@ #include "util.h" #include "vlog.h" -#define THIS_MODULE VLM_ovsdb_error -#include "vlog.h" +VLOG_DEFINE_THIS_MODULE(ovsdb_error); struct ovsdb_error { const char *tag; /* String for "error" member. */ @@ -87,7 +86,7 @@ ovsdb_syntax_error(const struct json *json, const char *tag, if (json) { /* XXX this is much too much information in some cases */ - error->syntax = json_to_string(json, 0); + error->syntax = json_to_string(json, JSSF_SORT); } return error; @@ -115,8 +114,16 @@ ovsdb_wrap_error(struct ovsdb_error *error, const char *details, ...) return error; } +/* Returns an ovsdb_error that represents an internal error for file name + * 'file' and line number 'line', with 'details' (formatted as with printf()) + * as the associated message. The caller is responsible for freeing the + * returned error. + * + * If 'inner_error' is nonnull then the returned error is wrapped around + * 'inner_error'. Takes ownership of 'inner_error'. */ struct ovsdb_error * -ovsdb_internal_error(const char *file, int line, const char *details, ...) +ovsdb_internal_error(struct ovsdb_error *inner_error, + const char *file, int line, const char *details, ...) { struct ds ds = DS_EMPTY_INITIALIZER; struct backtrace backtrace; @@ -145,6 +152,14 @@ ovsdb_internal_error(const char *file, int line, const char *details, ...) ds_put_format(&ds, " (%s %s%s)", program_name, VERSION, BUILDNR); + if (inner_error) { + char *s = ovsdb_error_to_string(inner_error); + ds_put_format(&ds, " (generated from: %s)", s); + free(s); + + ovsdb_error_destroy(inner_error); + } + error = ovsdb_error("internal error", "%s", ds_cstr(&ds)); ds_destroy(&ds); @@ -238,4 +253,3 @@ ovsdb_error_assert(struct ovsdb_error *error) ovsdb_error_destroy(error); } } -