linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, apusaka@google.com
Subject: RE: [Bluez] gatt: Use the same crypto instance for each device
Date: Fri, 15 Jan 2021 05:37:33 -0800 (PST)	[thread overview]
Message-ID: <60019a9d.1c69fb81.9fa78.c8eb@mx.google.com> (raw)
In-Reply-To: <20210115202432.Bluez.1.I608cbc9e4ad3ffb910de766da79c0f418ab1ba35@changeid>

[-- Attachment #1: Type: text/plain, Size: 9515 bytes --]

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=415407

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - FAIL
Output:
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:17:19: error: redundant redeclaration of ‘bt_crypto_new’ [-Werror=redundant-decls]
   17 | struct bt_crypto *bt_crypto_new(void);
      |                   ^~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:17:19: note: previous declaration of ‘bt_crypto_new’ was here
   17 | struct bt_crypto *bt_crypto_new(void);
      |                   ^~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:19:19: error: redundant redeclaration of ‘bt_crypto_ref’ [-Werror=redundant-decls]
   19 | struct bt_crypto *bt_crypto_ref(struct bt_crypto *crypto);
      |                   ^~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:19:19: note: previous declaration of ‘bt_crypto_ref’ was here
   19 | struct bt_crypto *bt_crypto_ref(struct bt_crypto *crypto);
      |                   ^~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:20:6: error: redundant redeclaration of ‘bt_crypto_unref’ [-Werror=redundant-decls]
   20 | void bt_crypto_unref(struct bt_crypto *crypto);
      |      ^~~~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:20:6: note: previous declaration of ‘bt_crypto_unref’ was here
   20 | void bt_crypto_unref(struct bt_crypto *crypto);
      |      ^~~~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:22:6: error: redundant redeclaration of ‘bt_crypto_random_bytes’ [-Werror=redundant-decls]
   22 | bool bt_crypto_random_bytes(struct bt_crypto *crypto,
      |      ^~~~~~~~~~~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:22:6: note: previous declaration of ‘bt_crypto_random_bytes’ was here
   22 | bool bt_crypto_random_bytes(struct bt_crypto *crypto,
      |      ^~~~~~~~~~~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:25:6: error: redundant redeclaration of ‘bt_crypto_e’ [-Werror=redundant-decls]
   25 | bool bt_crypto_e(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:25:6: note: previous declaration of ‘bt_crypto_e’ was here
   25 | bool bt_crypto_e(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:27:6: error: redundant redeclaration of ‘bt_crypto_ah’ [-Werror=redundant-decls]
   27 | bool bt_crypto_ah(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:27:6: note: previous declaration of ‘bt_crypto_ah’ was here
   27 | bool bt_crypto_ah(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:29:6: error: redundant redeclaration of ‘bt_crypto_c1’ [-Werror=redundant-decls]
   29 | bool bt_crypto_c1(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:29:6: note: previous declaration of ‘bt_crypto_c1’ was here
   29 | bool bt_crypto_c1(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:34:6: error: redundant redeclaration of ‘bt_crypto_s1’ [-Werror=redundant-decls]
   34 | bool bt_crypto_s1(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:34:6: note: previous declaration of ‘bt_crypto_s1’ was here
   34 | bool bt_crypto_s1(struct bt_crypto *crypto, const uint8_t k[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:37:6: error: redundant redeclaration of ‘bt_crypto_f4’ [-Werror=redundant-decls]
   37 | bool bt_crypto_f4(struct bt_crypto *crypto, uint8_t u[32], uint8_t v[32],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:37:6: note: previous declaration of ‘bt_crypto_f4’ was here
   37 | bool bt_crypto_f4(struct bt_crypto *crypto, uint8_t u[32], uint8_t v[32],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:39:6: error: redundant redeclaration of ‘bt_crypto_f5’ [-Werror=redundant-decls]
   39 | bool bt_crypto_f5(struct bt_crypto *crypto, uint8_t w[32], uint8_t n1[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:39:6: note: previous declaration of ‘bt_crypto_f5’ was here
   39 | bool bt_crypto_f5(struct bt_crypto *crypto, uint8_t w[32], uint8_t n1[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:42:6: error: redundant redeclaration of ‘bt_crypto_f6’ [-Werror=redundant-decls]
   42 | bool bt_crypto_f6(struct bt_crypto *crypto, uint8_t w[16], uint8_t n1[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:42:6: note: previous declaration of ‘bt_crypto_f6’ was here
   42 | bool bt_crypto_f6(struct bt_crypto *crypto, uint8_t w[16], uint8_t n1[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:45:6: error: redundant redeclaration of ‘bt_crypto_g2’ [-Werror=redundant-decls]
   45 | bool bt_crypto_g2(struct bt_crypto *crypto, uint8_t u[32], uint8_t v[32],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:45:6: note: previous declaration of ‘bt_crypto_g2’ was here
   45 | bool bt_crypto_g2(struct bt_crypto *crypto, uint8_t u[32], uint8_t v[32],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:47:6: error: redundant redeclaration of ‘bt_crypto_h6’ [-Werror=redundant-decls]
   47 | bool bt_crypto_h6(struct bt_crypto *crypto, const uint8_t w[16],
      |      ^~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:47:6: note: previous declaration of ‘bt_crypto_h6’ was here
   47 | bool bt_crypto_h6(struct bt_crypto *crypto, const uint8_t w[16],
      |      ^~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:49:6: error: redundant redeclaration of ‘bt_crypto_sign_att’ [-Werror=redundant-decls]
   49 | bool bt_crypto_sign_att(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:49:6: note: previous declaration of ‘bt_crypto_sign_att’ was here
   49 | bool bt_crypto_sign_att(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:52:6: error: redundant redeclaration of ‘bt_crypto_verify_att_sign’ [-Werror=redundant-decls]
   52 | bool bt_crypto_verify_att_sign(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:52:6: note: previous declaration of ‘bt_crypto_verify_att_sign’ was here
   52 | bool bt_crypto_verify_att_sign(struct bt_crypto *crypto, const uint8_t key[16],
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from src/shared/gatt-db.c:25:
./src/shared/crypto.h:54:6: error: redundant redeclaration of ‘bt_crypto_gatt_hash’ [-Werror=redundant-decls]
   54 | bool bt_crypto_gatt_hash(struct bt_crypto *crypto, struct iovec *iov,
      |      ^~~~~~~~~~~~~~~~~~~
In file included from ./src/shared/gatt-db.h:11,
                 from src/shared/gatt-db.c:24:
./src/shared/crypto.h:54:6: note: previous declaration of ‘bt_crypto_gatt_hash’ was here
   54 | bool bt_crypto_gatt_hash(struct bt_crypto *crypto, struct iovec *iov,
      |      ^~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:6808: src/shared/gatt-db.lo] Error 1
make: *** [Makefile:4023: all] Error 2


##############################
Test: MakeCheck - SKIPPED
Output:
checkbuild not success



---
Regards,
Linux Bluetooth


  reply	other threads:[~2021-01-15 13:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-15 12:25 [Bluez PATCH] gatt: Use the same crypto instance for each device Archie Pusaka
2021-01-15 13:37 ` bluez.test.bot [this message]
2021-01-15 18:39 ` Luiz Augusto von Dentz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=60019a9d.1c69fb81.9fa78.c8eb@mx.google.com \
    --to=bluez.test.bot@gmail.com \
    --cc=apusaka@google.com \
    --cc=linux-bluetooth@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).