diff --git a/src/ncp/ncp_base_ftd.cpp b/src/ncp/ncp_base_ftd.cpp index cf846b406..cd1957ccb 100644 --- a/src/ncp/ncp_base_ftd.cpp +++ b/src/ncp/ncp_base_ftd.cpp @@ -1523,16 +1523,9 @@ exit: template <> otError NcpBase::HandlePropertyGet(void) { - otError error = OT_ERROR_NONE; - uint8_t header = SPINEL_HEADER_FLAG | SPINEL_HEADER_IID_0; otSrpServerState srpServerState = otSrpServerGetState(mInstance); - SuccessOrExit(error = mEncoder.BeginFrame(header, SPINEL_CMD_PROP_VALUE_IS, SPINEL_PROP_SRP_SERVER_ENABLED)); - SuccessOrExit(error = mEncoder.WriteBool(srpServerState != OT_SRP_SERVER_STATE_DISABLED)); - SuccessOrExit(error = mEncoder.EndFrame()); - -exit: - return error; + return mEncoder.WriteBool(srpServerState != OT_SRP_SERVER_STATE_DISABLED); } #if OPENTHREAD_CONFIG_BORDER_ROUTING_ENABLE @@ -1550,16 +1543,7 @@ exit: template <> otError NcpBase::HandlePropertyGet(void) { - otError error = OT_ERROR_NONE; - uint8_t header = SPINEL_HEADER_FLAG | SPINEL_HEADER_IID_0; - - SuccessOrExit(error = - mEncoder.BeginFrame(header, SPINEL_CMD_PROP_VALUE_IS, SPINEL_PROP_SRP_SERVER_AUTO_ENABLE_MODE)); - SuccessOrExit(error = mEncoder.WriteBool(otSrpServerIsAutoEnableMode(mInstance))); - SuccessOrExit(error = mEncoder.EndFrame()); - -exit: - return error; + return mEncoder.WriteBool(otSrpServerIsAutoEnableMode(mInstance)); } #endif // OPENTHREAD_CONFIG_BORDER_ROUTING_ENABLE #endif // OPENTHREAD_CONFIG_SRP_SERVER_ENABLE