core-lightning/common/errcode.h

#ifndef LIGHTNING_COMMON_ERRCODE_H
#define LIGHTNING_COMMON_ERRCODE_H

#include "config.h"

#include <ccan/short_types/short_types.h>

// Setup errors
#define EXITCODE_SUBDAEMON_FAIL 10
#define EXITCODE_PIDFILE_LOCK 11

// HSM errors code
#define EXITCODE_HSM_GENERIC_ERROR 20
#define EXITCODE_HSM_ERROR_IS_ENCRYPT 21
#define EXITCODE_HSM_BAD_PASSWORD 22
#define EXITCODE_HSM_PASSWORD_INPUT_ERR 23
#define EXITCODE_ERROR_HSM_FILE 24

// Wallet error
#define EXITCODE_WALLET_DB_MISMATCH 30

#endif /* LIGHTNING_COMMON_ERRCODE_H */
Use dedicated type for error codes Before this patch we used `int` for error codes. The problem with `int` is that we try to pass it to/from wire and the size of `int` is not defined by the standard. So a sender with 4-byte `int` would write 4 bytes to the wire and a receiver with 2-byte `int` (for example) would read just 2 bytes from the wire. To resolve this: * Introduce an error code type with a known size: `typedef s32 errcode_t`. * Change all error code macros to constants of type `errcode_t`. Constants also play better with gdb - it would visualize the name of the constant instead of the numeric value. * Change all functions that take error codes to take the new type `errcode_t` instead of `int`. * Introduce towire / fromwire functions to send / receive the newly added type `errcode_t` and use it instead of `towire_int()`. In addition: * Remove the now unneeded `towire_int()`. * Replace a hardcoded error code `-2` with a new constant `INVOICE_EXPIRED_DURING_WAIT` (903). Changelog-Changed: The waitinvoice command would now return error code 903 to designate that the invoice expired during wait, instead of the previous -2 2020-01-26 13:52:29 +01:00			`#ifndef LIGHTNING_COMMON_ERRCODE_H`
			`#define LIGHTNING_COMMON_ERRCODE_H`

			`#include "config.h"`

			`#include <ccan/short_types/short_types.h>`

lightningd: expand exit codes for various failures. Most unexpected ones are still 1, but there are a few recognizable error codes worth documenting. Rename the HSM ones to put ERRCODE_ at the front, since we have non-HSM ones too now. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> 2022-07-20 04:29:25 +02:00			`// Setup errors`
			`#define EXITCODE_SUBDAEMON_FAIL 10`
			`#define EXITCODE_PIDFILE_LOCK 11`

lightningd: support hsm error code Suggested-by: Rusty Russell Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com> Changelog-Changed: Support hsm specific error error code in lightning-cli 2021-12-14 22:57:21 +01:00			`// HSM errors code`
lightningd: expand exit codes for various failures. Most unexpected ones are still 1, but there are a few recognizable error codes worth documenting. Rename the HSM ones to put ERRCODE_ at the front, since we have non-HSM ones too now. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> 2022-07-20 04:29:25 +02:00			`#define EXITCODE_HSM_GENERIC_ERROR 20`
			`#define EXITCODE_HSM_ERROR_IS_ENCRYPT 21`
			`#define EXITCODE_HSM_BAD_PASSWORD 22`
			`#define EXITCODE_HSM_PASSWORD_INPUT_ERR 23`
			`#define EXITCODE_ERROR_HSM_FILE 24`

			`// Wallet error`
			`#define EXITCODE_WALLET_DB_MISMATCH 30`
lightningd: support hsm error code Suggested-by: Rusty Russell Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com> Changelog-Changed: Support hsm specific error error code in lightning-cli 2021-12-14 22:57:21 +01:00
Use dedicated type for error codes Before this patch we used `int` for error codes. The problem with `int` is that we try to pass it to/from wire and the size of `int` is not defined by the standard. So a sender with 4-byte `int` would write 4 bytes to the wire and a receiver with 2-byte `int` (for example) would read just 2 bytes from the wire. To resolve this: * Introduce an error code type with a known size: `typedef s32 errcode_t`. * Change all error code macros to constants of type `errcode_t`. Constants also play better with gdb - it would visualize the name of the constant instead of the numeric value. * Change all functions that take error codes to take the new type `errcode_t` instead of `int`. * Introduce towire / fromwire functions to send / receive the newly added type `errcode_t` and use it instead of `towire_int()`. In addition: * Remove the now unneeded `towire_int()`. * Replace a hardcoded error code `-2` with a new constant `INVOICE_EXPIRED_DURING_WAIT` (903). Changelog-Changed: The waitinvoice command would now return error code 903 to designate that the invoice expired during wait, instead of the previous -2 2020-01-26 13:52:29 +01:00			`#endif /* LIGHTNING_COMMON_ERRCODE_H */`