# GnuTLS -- Information about our contribution rules and coding style Anyone is welcome to contribute to GnuTLS. You can either take up tasks from our [planned list](https://gitlab.com/gnutls/gnutls/milestones), or suprise us with enhancement we didn't plan for. In all cases be prepared to defend and justify your enhancements, and get through few rounds of changes. We try to stick to the following rules, so when contributing please try to follow them too. # Git commits: Note that when contributing code you will need to assert that the contribution is in accordance to the "Developer's Certificate of Origin" as found in the file [DCO.txt](doc/DCO.txt). To indicate that, make sure that your contributions (patches or merge requests), contain a "Signed-off-by" line, with your real name and e-mail address. To automate the process use "git am -s" to produce patches and/or set the a template to simplify this process, as follows. ``` $ cp devel/git-template ~/.git-template [edit] $ git config commit.template ~/.git-template ``` # Test suite: New functionality should be accompanied by a test case which verifies the correctness of GnuTLS' operation on successful use of the new functionality, as well as on fail cases. The GnuTLS test suite is run on "make check" on every system GnuTLS is installed, except for the tests/suite part which is only run during development. For testing functionality of gnutls we use two test unit testing frameworks: 1. The gnutls testing framework as in [utils.h](tests/utils.h), usually for high level tests such as testing a client against a server. See [set_x509_key_mem.c](tests/set_x509_key_mem.c). 2. The cmocka unit testing framework, for unit testing of functions or interfaces. See [dtls-sliding-window.c](tests/dtls-sliding-window.c). Certificates for testing purposes are available at [cert-common.h](tests/cert-common.h). Note that we do not regenerate test certificates when they expire, but we rather fix the test's time using datefudge or gnutls_global_set_time_function(). For example, see [x509cert-tl.c](tests/x509cert-tl.c). # File names: Files are split to directories according to the subsystem they belong to. Examples are x509/, minitasn1/, openpgp/, opencdk/ etc. The files in the root directory related to the main TLS protocol implementation. # C dialect: While parts of GnuTLS were written in older dialects, new code in GnuTLS are expected to conform to C99. Exceptions could be made for C99 features that are not supported in popular platforms on a case by case basis. # Indentation style: In general, use the Linux kernel coding style. You may indent the source using GNU indent, e.g. "indent -linux *.c". # Function names: All the function names use underscore ```_```, to separate words, functions like ```gnutlsDoThat``` are not used. The exported function names usually start with the ```gnutls_``` prefix, and the words that follow specify the exact subsystem of gnutls that this function refers to. E.g. ```gnutls_x509_crt_get_dn```, refers to the X.509 certificate parsing part of gnutls. Some of the used prefixes are the following. * ```gnutls_x509_crt_``` for the X.509 certificate part * ```gnutls_session_``` for the TLS session part (but this may be omited) * ```gnutls_handshake_``` for the TLS handshake part * ```gnutls_record_``` for the TLS record protocol part * ```gnutls_alert_``` for the TLS alert protocol part * ```gnutls_credentials_``` for the credentials structures * ```gnutls_global_``` for the global structures handling Internal functions -- that are not exported in the API -- should be prefixed with an underscore. E.g. ```_gnutls_handshake_begin()``` Internal structures should not be exported. Especially pointers to internal data. Doing so harms future reorganization/rewrite of subsystems. All exported functions must be listed in libgnutls.map.in, in order to be exported. # Introducing new functions / API Prior to introducing any new API consider all options to offer the same functionality without introducing a new function. The reason is that we want to avoid breaking the ABI, and thus we cannot typically remove any function that was added (though we have few exceptions). Since we cannot remove, it means that experimental APIs, or helper APIs that are not typically needed may become a burden to maintain in the future. That is, they may prevent a refactoring, or require to keep legacy code. As such, some questions to answer before adding a new API: * Is this API useful for a large class of applications, or is it limited to few? * If it is limited to few, can we work around that functionality without a new API? * Would that function be relevant in the future when a new protocol such TLS 13.0 is made available? Would it harm the addition of a new protocol? # Constructed types: The constructed types in gnutls always have the ```gnutls_``` prefix. Definitions, value defaults and enumerated values should be in capitals. E.g. ```GNUTLS_CIPHER_3DES_CBC```. Structures should have the ```_st``` suffix in their name even if they are a typedef. One can use the sizeof() on types with ```_st``` as suffix to get the structure's size. Other constructed types should have the ```_t``` suffix. A pointer to a structure also has the ```_t``` suffix. # Function parameters: The gnutls functions accept parameters in the order: 1. Input parameters 2. Output parameters When data and size is expected, a gnutls_datum structure should be used (or more precisely a pointer to the structure). # Callback function parameters: Callback functions should be avoided, if this is possible. Callbacks that refer to a TLS session should include the current session as a parameter, in order for the called function to be able to retrieve the data associated with the session. This is not always done though -- see the push/pull callbacks. # Return values: Functions in gnutls return an int type, when possible. In that case 0 should be returned in case of success, or maybe a positive value, if some other indication is needed. A negative value always indicates failure. All the available error codes are defined in gnutls.h and a description is available in gnutls_errors.c Functions which are intended to return a boolean value should return a type of bool, and it is recommended to contain the string '_is_' on its function name; e.g., ``` bool _gnutls_is_not_prehashed(); ``` That allows the distinguishing functions that return negative errors from boolean functions to both the developer and the compiler. Note that in the past the 'unsigned' type was used to distinguish boolean functions and several of these still exist. # Usage of assert() The assert() macro --not to be confused with gnutls_assert()-- is used exceptionally on impossible situations to assist static analysis tools. That is, it should be used when the static analyzer used in CI (currently clang analyzer), detects an error which is on an impossible situation. In these cases assert() is used to rule out that case. For example in the situation where a pointer is known to be non-null, but the static analyzer cannot rule it out, we use code like the following: ``` assert(ptr != NULL); ptr->deref = 3; ``` Since GnuTLS is a library no other uses of assert() macro are acceptable. The NDEBUG macro is not used in GnuTLS compilation, so the assert() macros are always active. # Symbol and library versioning The library uses the libtool versioning system, which in turn results to a soname bump on incompatible changes. That is described in [hooks.m4](m4/hooks.m4). Despite its complexity that system is only sufficient to distinguish between versions of the library that have broke ABI (i.e., soname bump occurred). Today however, soname versioning isn't sufficient. Symbol versioning as provided by [libgnutls.map](lib/libgnutls.map) have several advantages. * they allow for symbol clashing between different gnutls library versions being in the same address space. * they allow installers to detect the library version used for an application utilizing a specific symbol * the allow introducing multiple versions of a symbol a la libc, keeping the semantics of old functions while introducing new. As such for every symbol introduced on a particular version, we create an entry in libgnutls.map based on the version and containing the new symbols. For example, if in version 3.6.2 we introduce symbol ```gnutls_xyz```, the entry would be: GNUTLS_3_6_2 { global: gnutls_xyz; } GNUTLS_3_6_1; where ```GNUTLS_3_6_1``` is the last version that symbols were introduced, and indicates a dependency of 3.6.2 to symbols of 3.6.1. Note that when the soname version is bumped, i.e., the ABI is broken all the previous symbol versions should be combined into a single. For example on the 3.4.0 soname bump, all symbols were put under the GNUTLS_3_4 version. Backporting new symbols to an old version which is soname compatible is not allowed (can cause quite some problems). Backporting symbols to an incompatible soname version is allowed, but must ensure that the symbol version used for the backported symbol version is distinct from the original library symbol version. E.g., if symbol ```gnutls_xyz``` with version GNUTLS_3_6_3 is backported on gnutls 3.3.15, it should use version GNUTLS_3_3_15. # Auto-generated files: Several parts of the documentation and the command line tools parameters files (.def) are auto-generated. Normally when introducing new functions, or adding new command line options to tools you need to run 'make files-update', review the output (when feasible) and commit it separately, e.g., with a message: "auto-generated files update". # Guile bindings: Parts of the Guile bindings, such as types (aka. "SMOBs"), enum values, constants, are automatically generated. This is handled by the modules under `guile/modules/gnutls/build/'; these modules are only used at build-time and are not installed. The Scheme variables they generate (e.g., constants, type predicates, etc.) are exported to user programs through `gnutls.scm' and `gnutls/extra.scm', both of which are installed. For instance, when adding/removing/renaming enumerates or constants, two things must be done: 1. Update the enum list in `build/enums.scm' (currently dependencies are not tracked, so you have to run "make clean all" in `guile/' after). 2. Update the export list of `gnutls.scm' (or `extra.scm'). Note that, for constants and enums, "schemefied" names are used, as noted under the "Guile API Conventions" node of the manual.