From 2c41df5bf8327f3bc6ee25cf2aa570bb159cb348 Mon Sep 17 00:00:00 2001 From: Chris Dunlap Date: Wed, 14 Jan 2015 16:31:21 -0800 Subject: [PATCH] Cleanup _zed_event_add_nvpair() When _zed_event_add_var() was updated to be the common routine for adding zedlet environment variables, an additional snprintf() was added to the processing of each nvpair. This commit changes _zed_event_add_nvpair() to directly call _zed_event_add_var() for nvpair non-array types, thereby removing a superfluous call to snprintf(). For consistency, the helper functions for converting nvpair array types are similarly adjusted to add variables. The _zed_event_value_is_hex() and _zed_event_add_var() functions have been moved up in the file since forward declarations are not used, but no changes have been made to these functions. Signed-off-by: Chris Dunlap Signed-off-by: Brian Behlendorf Closes #3042 --- cmd/zed/zed_event.c | 495 +++++++++++++++++++++++--------------------- 1 file changed, 263 insertions(+), 232 deletions(-) diff --git a/cmd/zed/zed_event.c b/cmd/zed/zed_event.c index 6f6c5ba42c5a..4dc752b6923a 100644 --- a/cmd/zed/zed_event.c +++ b/cmd/zed/zed_event.c @@ -159,370 +159,408 @@ zed_event_seek(struct zed_conf *zcp, uint64_t saved_eid, int64_t saved_etime[]) return (found ? 0 : -1); } +/* + * Return non-zero if nvpair [name] should be formatted in hex; o/w, return 0. + */ +static int +_zed_event_value_is_hex(const char *name) +{ + const char *hex_suffix[] = { + "_guid", + "_guids", + NULL + }; + const char **pp; + char *p; + + if (!name) + return (0); + + for (pp = hex_suffix; *pp; pp++) { + p = strstr(name, *pp); + if (p && strlen(p) == strlen(*pp)) + return (1); + } + return (0); +} + +/* + * Add an environment variable for [eid] to the container [zsp]. + * + * The variable name is the concatenation of [prefix] and [name] converted to + * uppercase with non-alphanumeric characters converted to underscores; + * [prefix] is optional, and [name] must begin with an alphabetic character. + * If the converted variable name already exists within the container [zsp], + * its existing value will be replaced with the new value. + * + * The variable value is specified by the format string [fmt]. + * + * Returns 0 on success, and -1 on error (with errno set). + * + * All environment variables in [zsp] should be added through this function. + */ +static int +_zed_event_add_var(uint64_t eid, zed_strings_t *zsp, + const char *prefix, const char *name, const char *fmt, ...) +{ + char keybuf[MAXBUF]; + char valbuf[MAXBUF]; + char *dstp; + const char *srcp; + const char *lastp; + int n; + int buflen; + va_list vargs; + + assert(zsp != NULL); + assert(fmt != NULL); + + if (!name) { + errno = EINVAL; + zed_log_msg(LOG_WARNING, + "Failed to add variable for eid=%llu: Name is empty", eid); + return (-1); + } else if (!isalpha(name[0])) { + errno = EINVAL; + zed_log_msg(LOG_WARNING, + "Failed to add variable for eid=%llu: " + "Name \"%s\" is invalid", eid, name); + return (-1); + } + /* + * Construct the string key by converting PREFIX (if present) and NAME. + */ + dstp = keybuf; + lastp = keybuf + sizeof (keybuf); + if (prefix) { + for (srcp = prefix; *srcp && (dstp < lastp); srcp++) + *dstp++ = isalnum(*srcp) ? toupper(*srcp) : '_'; + } + for (srcp = name; *srcp && (dstp < lastp); srcp++) + *dstp++ = isalnum(*srcp) ? toupper(*srcp) : '_'; + + if (dstp == lastp) { + errno = ENAMETOOLONG; + zed_log_msg(LOG_WARNING, + "Failed to add variable for eid=%llu: Name too long", eid); + return (-1); + } + *dstp = '\0'; + /* + * Construct the string specified by "[PREFIX][NAME]=[FMT]". + */ + dstp = valbuf; + buflen = sizeof (valbuf); + n = strlcpy(dstp, keybuf, buflen); + if (n >= sizeof (valbuf)) { + errno = EMSGSIZE; + zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", + keybuf, eid, "Exceeded buffer size"); + return (-1); + } + dstp += n; + buflen -= n; + + *dstp++ = '='; + buflen--; + + va_start(vargs, fmt); + n = vsnprintf(dstp, buflen, fmt, vargs); + va_end(vargs); + + if ((n < 0) || (n >= buflen)) { + errno = EMSGSIZE; + zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", + keybuf, eid, "Exceeded buffer size"); + return (-1); + } else if (zed_strings_add(zsp, keybuf, valbuf) < 0) { + zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", + keybuf, eid, strerror(errno)); + return (-1); + } + return (0); +} + +static int +_zed_event_add_array_err(uint64_t eid, const char *name) +{ + errno = EMSGSIZE; + zed_log_msg(LOG_WARNING, + "Failed to convert nvpair \"%s\" for eid=%llu: " + "Exceeded buffer size", name, eid); + return (-1); +} + static int -_zed_event_convert_int8_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_int8_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; int8_t *i8p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_INT8_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_int8_array(nvp, &i8p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%d ", i8p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_uint8_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_uint8_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; uint8_t *u8p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_UINT8_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_uint8_array(nvp, &u8p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%u ", u8p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_int16_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_int16_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; int16_t *i16p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_INT16_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_int16_array(nvp, &i16p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%d ", i16p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_uint16_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_uint16_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; uint16_t *u16p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_UINT16_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_uint16_array(nvp, &u16p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%u ", u16p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_int32_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_int32_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; int32_t *i32p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_INT32_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_int32_array(nvp, &i32p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%d ", i32p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_uint32_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_uint32_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; uint32_t *u32p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_UINT32_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_uint32_array(nvp, &u32p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%u ", u32p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_int64_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_int64_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; int64_t *i64p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_INT64_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_int64_array(nvp, &i64p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%lld ", (u_longlong_t) i64p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_uint64_array(char *buf, int buflen, nvpair_t *nvp, - const char *fmt) +_zed_event_add_uint64_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; + const char *fmt; uint64_t *u64p; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_UINT64_ARRAY)); + name = nvpair_name(nvp); + fmt = _zed_event_value_is_hex(name) ? "0x%.16llX " : "%llu "; (void) nvpair_value_uint64_array(nvp, &u64p, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, fmt, (u_longlong_t) u64p[i]); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } static int -_zed_event_convert_string_array(char *buf, int buflen, nvpair_t *nvp) +_zed_event_add_string_array(uint64_t eid, zed_strings_t *zsp, + const char *prefix, nvpair_t *nvp) { + char buf[MAXBUF]; + int buflen = sizeof (buf); + const char *name; char **strp; uint_t nelem; uint_t i; char *p; int n; - assert(buf != NULL); + assert((nvp != NULL) && (nvpair_type(nvp) == DATA_TYPE_STRING_ARRAY)); + name = nvpair_name(nvp); (void) nvpair_value_string_array(nvp, &strp, &nelem); for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) { n = snprintf(p, buflen, "%s ", strp[i] ? strp[i] : ""); - if ((n < 0) || (n >= buflen)) { - *buf = '\0'; - return (-1); - } + if ((n < 0) || (n >= buflen)) + return (_zed_event_add_array_err(eid, name)); p += n; buflen -= n; } if (nelem > 0) *--p = '\0'; - return (p - buf); -} - -/* - * Return non-zero if nvpair [name] should be formatted in hex; o/w, return 0. - */ -static int -_zed_event_value_is_hex(const char *name) -{ - const char *hex_suffix[] = { - "_guid", - "_guids", - NULL - }; - const char **pp; - char *p; - - if (!name) - return (0); - - for (pp = hex_suffix; *pp; pp++) { - p = strstr(name, *pp); - if (p && strlen(p) == strlen(*pp)) - return (1); - } - return (0); -} - -/* - * Add an environment variable for [eid] to the container [zsp]. - * - * The variable name is the concatenation of [prefix] and [name] converted to - * uppercase with non-alphanumeric characters converted to underscores; - * [prefix] is optional, and [name] must begin with an alphabetic character. - * If the converted variable name already exists within the container [zsp], - * its existing value will be replaced with the new value. - * - * The variable value is specified by the format string [fmt]. - * - * Returns 0 on success, and -1 on error (with errno set). - * - * All environment variables in [zsp] should be added through this function. - */ -static int -_zed_event_add_var(uint64_t eid, zed_strings_t *zsp, - const char *prefix, const char *name, const char *fmt, ...) -{ - char keybuf[MAXBUF]; - char valbuf[MAXBUF]; - char *dstp; - const char *srcp; - const char *lastp; - int n; - int buflen; - va_list vargs; - - assert(zsp != NULL); - assert(fmt != NULL); - - if (!name) { - errno = EINVAL; - zed_log_msg(LOG_WARNING, - "Failed to add variable for eid=%llu: Name is empty", eid); - return (-1); - } else if (!isalpha(name[0])) { - errno = EINVAL; - zed_log_msg(LOG_WARNING, - "Failed to add variable for eid=%llu: " - "Name \"%s\" is invalid", eid, name); - return (-1); - } - /* - * Construct the string key by converting PREFIX (if present) and NAME. - */ - dstp = keybuf; - lastp = keybuf + sizeof (keybuf); - if (prefix) { - for (srcp = prefix; *srcp && (dstp < lastp); srcp++) - *dstp++ = isalnum(*srcp) ? toupper(*srcp) : '_'; - } - for (srcp = name; *srcp && (dstp < lastp); srcp++) - *dstp++ = isalnum(*srcp) ? toupper(*srcp) : '_'; - - if (dstp == lastp) { - errno = ENAMETOOLONG; - zed_log_msg(LOG_WARNING, - "Failed to add variable for eid=%llu: Name too long", eid); - return (-1); - } - *dstp = '\0'; - /* - * Construct the string specified by "[PREFIX][NAME]=[FMT]". - */ - dstp = valbuf; - buflen = sizeof (valbuf); - n = strlcpy(dstp, keybuf, buflen); - if (n >= sizeof (valbuf)) { - errno = EMSGSIZE; - zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", - keybuf, eid, "Exceeded buffer size"); - return (-1); - } - dstp += n; - buflen -= n; - - *dstp++ = '='; - buflen--; - - va_start(vargs, fmt); - n = vsnprintf(dstp, buflen, fmt, vargs); - va_end(vargs); - - if ((n < 0) || (n >= buflen)) { - errno = EMSGSIZE; - zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", - keybuf, eid, "Exceeded buffer size"); - return (-1); - } else if (zed_strings_add(zsp, keybuf, valbuf) < 0) { - zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s", - keybuf, eid, strerror(errno)); - return (-1); - } - return (0); + return (_zed_event_add_var(eid, zsp, prefix, name, "%s", buf)); } /* @@ -537,10 +575,7 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp) { const char *name; data_type_t type; - char buf[MAXBUF]; - int n; - const char *fmt; - + const char *prefix = ZEVENT_VAR_PREFIX; boolean_t b; double d; uint8_t i8; @@ -557,119 +592,115 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp) switch (type) { case DATA_TYPE_BOOLEAN: - n = snprintf(buf, sizeof (buf), "%s", "1"); + _zed_event_add_var(eid, zsp, prefix, name, "%s", "1"); break; case DATA_TYPE_BOOLEAN_VALUE: (void) nvpair_value_boolean_value(nvp, &b); - n = snprintf(buf, sizeof (buf), "%s", b ? "1" : "0"); + _zed_event_add_var(eid, zsp, prefix, name, "%s", b ? "1" : "0"); break; case DATA_TYPE_BYTE: (void) nvpair_value_byte(nvp, &i8); - n = snprintf(buf, sizeof (buf), "%d", i8); + _zed_event_add_var(eid, zsp, prefix, name, "%d", i8); break; case DATA_TYPE_INT8: (void) nvpair_value_int8(nvp, (int8_t *) &i8); - n = snprintf(buf, sizeof (buf), "%d", i8); + _zed_event_add_var(eid, zsp, prefix, name, "%d", i8); break; case DATA_TYPE_UINT8: (void) nvpair_value_uint8(nvp, &i8); - n = snprintf(buf, sizeof (buf), "%u", i8); + _zed_event_add_var(eid, zsp, prefix, name, "%u", i8); break; case DATA_TYPE_INT16: (void) nvpair_value_int16(nvp, (int16_t *) &i16); - n = snprintf(buf, sizeof (buf), "%d", i16); + _zed_event_add_var(eid, zsp, prefix, name, "%d", i16); break; case DATA_TYPE_UINT16: (void) nvpair_value_uint16(nvp, &i16); - n = snprintf(buf, sizeof (buf), "%u", i16); + _zed_event_add_var(eid, zsp, prefix, name, "%u", i16); break; case DATA_TYPE_INT32: (void) nvpair_value_int32(nvp, (int32_t *) &i32); - n = snprintf(buf, sizeof (buf), "%d", i32); + _zed_event_add_var(eid, zsp, prefix, name, "%d", i32); break; case DATA_TYPE_UINT32: (void) nvpair_value_uint32(nvp, &i32); - n = snprintf(buf, sizeof (buf), "%u", i32); + _zed_event_add_var(eid, zsp, prefix, name, "%u", i32); break; case DATA_TYPE_INT64: (void) nvpair_value_int64(nvp, (int64_t *) &i64); - n = snprintf(buf, sizeof (buf), "%lld", (longlong_t) i64); + _zed_event_add_var(eid, zsp, prefix, name, + "%lld", (longlong_t) i64); break; case DATA_TYPE_UINT64: (void) nvpair_value_uint64(nvp, &i64); - fmt = _zed_event_value_is_hex(name) ? "0x%.16llX" : "%llu"; - n = snprintf(buf, sizeof (buf), fmt, (u_longlong_t) i64); + _zed_event_add_var(eid, zsp, prefix, name, + (_zed_event_value_is_hex(name) ? "0x%.16llX" : "%llu"), + (u_longlong_t) i64); break; case DATA_TYPE_DOUBLE: (void) nvpair_value_double(nvp, &d); - n = snprintf(buf, sizeof (buf), "%g", d); + _zed_event_add_var(eid, zsp, prefix, name, "%g", d); break; case DATA_TYPE_HRTIME: (void) nvpair_value_hrtime(nvp, (hrtime_t *) &i64); - n = snprintf(buf, sizeof (buf), "%llu", (u_longlong_t) i64); + _zed_event_add_var(eid, zsp, prefix, name, + "%llu", (u_longlong_t) i64); break; case DATA_TYPE_NVLIST: - /* FIXME */ - n = snprintf(buf, sizeof (buf), "%s", "_NOT_IMPLEMENTED_"); + _zed_event_add_var(eid, zsp, prefix, name, + "%s", "_NOT_IMPLEMENTED_"); /* FIXME */ break; case DATA_TYPE_STRING: (void) nvpair_value_string(nvp, &str); - n = snprintf(buf, sizeof (buf), "%s", (str ? str : "")); + _zed_event_add_var(eid, zsp, prefix, name, + "%s", (str ? str : "")); break; case DATA_TYPE_BOOLEAN_ARRAY: - /* FIXME */ - n = snprintf(buf, sizeof (buf), "%s", "_NOT_IMPLEMENTED_"); + _zed_event_add_var(eid, zsp, prefix, name, + "%s", "_NOT_IMPLEMENTED_"); /* FIXME */ break; case DATA_TYPE_BYTE_ARRAY: - /* FIXME */ - n = snprintf(buf, sizeof (buf), "%s", "_NOT_IMPLEMENTED_"); + _zed_event_add_var(eid, zsp, prefix, name, + "%s", "_NOT_IMPLEMENTED_"); /* FIXME */ break; case DATA_TYPE_INT8_ARRAY: - n = _zed_event_convert_int8_array(buf, sizeof (buf), nvp); + _zed_event_add_int8_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_UINT8_ARRAY: - n = _zed_event_convert_uint8_array(buf, sizeof (buf), nvp); + _zed_event_add_uint8_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_INT16_ARRAY: - n = _zed_event_convert_int16_array(buf, sizeof (buf), nvp); + _zed_event_add_int16_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_UINT16_ARRAY: - n = _zed_event_convert_uint16_array(buf, sizeof (buf), nvp); + _zed_event_add_uint16_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_INT32_ARRAY: - n = _zed_event_convert_int32_array(buf, sizeof (buf), nvp); + _zed_event_add_int32_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_UINT32_ARRAY: - n = _zed_event_convert_uint32_array(buf, sizeof (buf), nvp); + _zed_event_add_uint32_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_INT64_ARRAY: - n = _zed_event_convert_int64_array(buf, sizeof (buf), nvp); + _zed_event_add_int64_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_UINT64_ARRAY: - fmt = _zed_event_value_is_hex(name) ? "0x%.16llX " : "%llu "; - n = _zed_event_convert_uint64_array(buf, sizeof (buf), - nvp, fmt); + _zed_event_add_uint64_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_STRING_ARRAY: - n = _zed_event_convert_string_array(buf, sizeof (buf), nvp); + _zed_event_add_string_array(eid, zsp, prefix, nvp); break; case DATA_TYPE_NVLIST_ARRAY: - /* FIXME */ - n = snprintf(buf, sizeof (buf), "%s", "_NOT_IMPLEMENTED_"); + _zed_event_add_var(eid, zsp, prefix, name, + "%s", "_NOT_IMPLEMENTED_"); /* FIXME */ break; default: + errno = EINVAL; zed_log_msg(LOG_WARNING, "Failed to convert nvpair \"%s\" for eid=%llu: " "Unrecognized type=%u", name, eid, (unsigned int) type); - return; - } - if ((n < 0) || (n >= sizeof (buf))) { - zed_log_msg(LOG_WARNING, - "Failed to convert nvpair \"%s\" for eid=%llu: %s", - name, eid, "Exceeded buffer size"); - return; + break; } - _zed_event_add_var(eid, zsp, ZEVENT_VAR_PREFIX, name, "%s", buf); } /*