[riot-notifications] [RIOT-OS/RIOT] FIDO2 support in RIOT (#16489)

Nils Ollrogge notifications at github.com
Fri May 28 14:32:29 CEST 2021


@Ollrogge commented on this pull request.



> +
+size_t fido2_ctap_handle_request(uint8_t *req, size_t size, ctap_resp_t *resp,
+                                 bool (*should_cancel)(void))
+{
+    CborEncoder encoder = { 0 };
+    uint8_t cmd = *req;
+    uint8_t *buf = resp->data;
+
+    req++;
+    size--;
+
+    cbor_encoder_init(&encoder, buf, CTAP_MAX_MSG_SIZE, 0);
+
+    /* handle common error case */
+    switch (cmd) {
+    case CTAP_MAKE_CREDENTIAL:

Running `uncrustify -c uncrustify-riot.cfg --no-backup sys/fido2/ctap/ctap.c` will unindent the switch statement. So I gues no indention is wanted ?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16489#discussion_r641511918
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210528/5091ecaa/attachment.htm>


More information about the notifications mailing list