[PATCH 1/4] gsm0480: RELEASE COMPLETE can be without any payload

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Neels Hofmeyr nhofmeyr at sysmocom.de
Tue Apr 26 11:28:43 UTC 2016


On Fri, Apr 22, 2016 at 02:41:12PM +0200, Sergey Kostanbaev wrote:
> From: Sergey Kostanbaev <Sergey.Kostanbaev at gmail.com>
> 
> ---
>  src/gsm/gsm0480.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/src/gsm/gsm0480.c b/src/gsm/gsm0480.c
> index 8963b78..55bddd5 100644
> --- a/src/gsm/gsm0480.c
> +++ b/src/gsm/gsm0480.c
> @@ -248,12 +248,6 @@ int gsm0480_decode_ss_request(const struct gsm48_hdr *hdr, uint16_t len,
>  				struct ss_request *req)
>  {
>  	int rc = 0;
> -
> -	if (len < sizeof(*hdr) + 2) {
> -		LOGP(0, LOGL_DEBUG, "SS Request is too short.\n");
> -		return 0;
> -	}
> -
>  	if (gsm48_hdr_pdisc(hdr) == GSM48_PDISC_NC_SS) {

I believe this should only remove the "+ 2" and still check for 

  if (len < sizeof(*hdr)) {
    return 0;
  }

since the code continues to dereference hdr, or is this the explicit
responsibility of the caller?

~Neels

-- 
- Neels Hofmeyr <nhofmeyr at sysmocom.de>          http://www.sysmocom.de/
=======================================================================
* sysmocom - systems for mobile communications GmbH
* Alt-Moabit 93
* 10559 Berlin, Germany
* Sitz / Registered office: Berlin, HRB 134158 B
* Geschäftsführer / Managing Directors: Holger Freyther, Harald Welte
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20160426/c0b2bfde/attachment.bin>


More information about the OpenBSC mailing list