Natt #15

Merged
merged 5 commits into from Apr 30, 2012

Conversation

Projects
None yet
2 participants
@arnatal
Member

arnatal commented Apr 30, 2012

  • Some draft minor details support.
  • Auth data check implementation.
@ljakab

This comment has been minimized.

Show comment
Hide comment
@ljakab

ljakab Apr 30, 2012

Member

Este commit necesitara unos cambios por mi parte tambien:
arnatal/lispmob@05f499b

Lo que devuelve la funcion es lo mismo para cuando haya un error
(default:) y en el caso de 0. En principio deberia funcionar sin tus
cambios, por que 0 iria a default. Pero como 0 es un caso valido tus
cambios tienen sentido, y habria cambiar el return code del default.
Pero en ese caso habra que comprobar todas las llamadas a esa funcion,
para ver si aguna llamada hace algo si recibe error o no.

Voy a hacer el pull tal cual, porque el funcionamiento actual no
cambiara, pero me apunto revisarlo una vez tengamos natt en 'testing'.

Saludos,
-Lori

On 04/30/12 18:33, Alberto Rodríguez Natal wrote:

  • Some draft minor details support.
  • Auth data check implementation.

You can merge this Pull Request by running:

git pull https://github.com/arnatal/lispmob natt

Or you can view, comment on it, or merge it online at:

#15

-- Commit Summary --

  • Added support for R bit and I bit in ECM Map Register messages.
  • Adapted Info-Request to the new draft (AFI = 0 instead of LCAF AFi).
  • Support for AFI 0 in get_addr_len (for AFI=0 in Private ETR RLOC in Info-Reply)
  • Fixed bug in R bit in ECM Map-Registers.
  • Auth data check in Info-reply.

-- File Changes --

M lispd/lispd.h (25)
M lispd/lispd_external.h (7)
M lispd/lispd_info_nat.c (2)
M lispd/lispd_info_reply.c (30)
M lispd/lispd_info_request.c (6)
M lispd/lispd_lib.c (3)
M lispd/lispd_nat_lib.c (83)

-- Patch Links --

https://github.com/LISPmob/lispmob/pull/15.patch
https://github.com/LISPmob/lispmob/pull/15.diff


Reply to this email directly or view it on GitHub:
#15

Member

ljakab commented Apr 30, 2012

Este commit necesitara unos cambios por mi parte tambien:
arnatal/lispmob@05f499b

Lo que devuelve la funcion es lo mismo para cuando haya un error
(default:) y en el caso de 0. En principio deberia funcionar sin tus
cambios, por que 0 iria a default. Pero como 0 es un caso valido tus
cambios tienen sentido, y habria cambiar el return code del default.
Pero en ese caso habra que comprobar todas las llamadas a esa funcion,
para ver si aguna llamada hace algo si recibe error o no.

Voy a hacer el pull tal cual, porque el funcionamiento actual no
cambiara, pero me apunto revisarlo una vez tengamos natt en 'testing'.

Saludos,
-Lori

On 04/30/12 18:33, Alberto Rodríguez Natal wrote:

  • Some draft minor details support.
  • Auth data check implementation.

You can merge this Pull Request by running:

git pull https://github.com/arnatal/lispmob natt

Or you can view, comment on it, or merge it online at:

#15

-- Commit Summary --

  • Added support for R bit and I bit in ECM Map Register messages.
  • Adapted Info-Request to the new draft (AFI = 0 instead of LCAF AFi).
  • Support for AFI 0 in get_addr_len (for AFI=0 in Private ETR RLOC in Info-Reply)
  • Fixed bug in R bit in ECM Map-Registers.
  • Auth data check in Info-reply.

-- File Changes --

M lispd/lispd.h (25)
M lispd/lispd_external.h (7)
M lispd/lispd_info_nat.c (2)
M lispd/lispd_info_reply.c (30)
M lispd/lispd_info_request.c (6)
M lispd/lispd_lib.c (3)
M lispd/lispd_nat_lib.c (83)

-- Patch Links --

https://github.com/LISPmob/lispmob/pull/15.patch
https://github.com/LISPmob/lispmob/pull/15.diff


Reply to this email directly or view it on GitHub:
#15

ljakab pushed a commit that referenced this pull request Apr 30, 2012

@ljakab ljakab merged commit 53ac8bf into LISPmob:natt Apr 30, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment