Skip to content

Commit

Permalink
Merge pull request #5 from hawkinswnaf/release-0.6.5.4
Browse files Browse the repository at this point in the history
keyring_open_with_pins() is gone (plus Makefile spiff-up) - Still needs additional testing.
  • Loading branch information
jheretic committed Aug 27, 2013
2 parents 6866370 + a624a46 commit ab9497a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
5 changes: 3 additions & 2 deletions lib/mdp/Makefile
Expand Up @@ -43,10 +43,11 @@ PLUGIN_VER = 0.1
TOPDIR = ../..
include $(TOPDIR)/Makefile.inc

SERVALD_ROOT=/home/hawkinsw/serval/serval-dna
SERVALD_ROOT?=/home/hawkinsw/serval/serval-dna
SERVALD_LIB_ROOT?=/home/hawkinsw/serval/serval-dna

CPPFLAGS+=-I$(SERVALD_ROOT)/ -I$(SERVALD_ROOT)/nacl/include
LIBS+=-L$(SERVALD_ROOT) -lservald
LIBS+=-L$(SERVALD_LIB_ROOT) -lserval

This comment has been minimized.

Copy link
@hawkinswnaf

hawkinswnaf Aug 29, 2013

Collaborator

This might have to be reworked. Expect a pull request to fix this.


default_target: $(PLUGIN_FULLNAME)

Expand Down
5 changes: 2 additions & 3 deletions lib/mdp/src/olsrd_mdp.c
Expand Up @@ -148,7 +148,7 @@ print_data(const char *label, const uint8_t *data, size_t len)
static void
mdp_checksum(uint8_t *data, const uint16_t data_len, uint8_t *hashbuf)
{
unsigned long long signature_bytes = SIGNATURE_BYTES;
int signature_bytes = SIGNATURE_BYTES;
print_data("Key", servald_key, servald_key_len);
print_data("Data", data, data_len);

Expand Down Expand Up @@ -1079,13 +1079,12 @@ timeout_timestamps(void *foo __attribute__ ((unused)))
static int
read_key_from_servald(const char *sid)
{
const char *pins = {"",};
unsigned char stowedSid[SID_SIZE];
unsigned char *found_private_key = NULL;
int found_private_key_len = 0;
int cn = 0, in = 0, kp = 0;

keyring = keyring_open_with_pins(pins);
keyring = keyring_open_instance();

This comment has been minimized.

Copy link
@hawkinswnaf

hawkinswnaf Aug 29, 2013

Collaborator

With the current version of serval-dna that we're using in the commotion-openwrt(/feed) build system, this change is unnecessary. Only the new(ish) versions of serval-dna have removed with_pins() and replaced it with _instance. To @danstaples, which version do we plan to ship with DR2?

This comment has been minimized.

Copy link
@dismantl

dismantl Aug 29, 2013

batphone-release-0.91

stowSid(stowedSid, 0, sid);

if (!keyring_find_sid(keyring, &cn, &in, &kp, stowedSid))
Expand Down

0 comments on commit ab9497a

Please sign in to comment.