Re: [ros-dev] [ros-diffs] [turner] 22683: expand %* to batch file params. b ug: #1629

[Murphy, Ged (Bolton)]

Brandon Turner wrote:

>   Index: batch.c
>  ===================================================================
>  --- batch.c    (revision 22683)
>  +++ batch.c    (working copy)
>  @@ -275,8 +275,17 @@
>       //
>       // Allocate enough memory to hold the params and copy them over 
>  without modifications
>       //
>  -    bc->raw_params = malloc(_tcslen(param));
>  -    _tcscpy(bc->raw_params,param);
>  +    bc->raw_params = malloc((_tcslen(param)+1) * sizeof(TCHAR));
>  +    if (bc->raw_params != NULL)
>  +    {
>  +        memset (bc->raw_params, 0, _tcslen(bc->raw_params) * 
>  sizeof(TCHAR));
>  +        _tcscpy(bc->raw_params,param);
>  +    }
>  +    else
>  +    {
>  +        error_out_of_memory();
>  +        return FALSE;
>  +    }
>   
>   #ifdef _DEBUG
>       DebugPrintf (_T("Batch: returns TRUE\n"));
>  
>  
>  look better?
>  
>  Brandon 

Yeah :)

Ged.