FreeCalypso > hg > fc-magnetite

view src/g23m-fad/app/bat_cfg.h @ 516:1ed9de6c90bd

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
src/g23m-gsm/sms/sms_for.c: bogus malloc removed The new error handling code that was not present in TCS211 blob version contains a malloc call that is bogus for 3 reasons: 1) The memory allocation in question is not needed in the first place; 2) libc malloc is used instead of one of the firmware's proper ways; 3) The memory allocation is made inside a function and then never freed, i.e., a memory leak. This bug was caught in gcc-built FreeCalypso fw projects (Citrine and Selenite) because our gcc environment does not allow any use of libc malloc (any reference to malloc produces a link failure), but this code from TCS3.2 is wrong even for Magnetite: if this code path is executed repeatedly over a long time, the many small allocations made by this malloc call without a subsequent free will eventually exhaust the malloc heap provided by the TMS470 environment, malloc will start returning NULL, and the bogus code will treat it as an error. Because the memory allocation in question is not needed at all, the fix entails simply removing it.
author Mychaela Falconia <falcon@freecalypso.org>
date Sun, 22 Jul 2018 06:04:49 +0000
parents 90eb61ecd093
children
line wrap: on
line source

/*
+-----------------------------------------------------------------------------
|  Project :  GSM-F&D (8411)
|  Modul   :  BAT library
+-----------------------------------------------------------------------------
|  Copyright 2005 Texas Instruments Berlin, AG
|                 All rights reserved.
|
|                 This file is confidential and a trade secret of Texas
|                 Instruments Berlin, AG
|                 The receipt of or possession of this file does not convey
|                 any rights to reproduce or disclose its contents or to
|                 manufacture, use, or sell anything it may describe, in
|                 whole, or in part, without the specific written consent of
|                 Texas Instruments Berlin, AG.
+-----------------------------------------------------------------------------
|  Purpose :  configuration for binary AT commands
+-----------------------------------------------------------------------------
*/
#ifndef BAT_CFG_H
#define BAT_CFG_H

#include "typedefs.h"   /* to include some defined types */
#include "l2p_types.h"
#include "gdd.h"


/*********************************************************************************
 *
 * defines
 *
 *********************************************************************************/

#define L2P_MAINTAIN_SIZE L2P_CONTROL_BLOCK_HDR_SIZE


/*********************************************************************************
 *
 * enumerations
 *
 *********************************************************************************/

typedef enum
{
  DEVICE_PACKET = 0
  /*more to be defined*/
}T_BAT_device;


/*********************************************************************************
 *
 * A description of the GDD interface
 *
 *********************************************************************************/

typedef struct
{
  T_GDD_FUNC                gdd_if; 
  T_GDD_CAP                 cap;
}T_BAT_adapter;


/*********************************************************************************
 *
 * A description of the L2P protocol Id
 *
 *********************************************************************************/

typedef struct
{
  T_L2P_SUPPORTED_PROTOCOLS protocol_id;
}T_BAT_l2p;

typedef struct
{
  T_BAT_device              device;
  T_BAT_adapter             adapter;
  T_BAT_l2p                 l2p;
}T_BAT_config;


/*********************************************************************************
 *
 * with this release the following configuration data should be used
 *
 *********************************************************************************/


extern T_BAT_config *BAT_APP_CONFIG;
void app_set_config (void);


#endif /* BAT_CFG_H */