[claudiuhks]

Originally Posted by K.K.Lv More clear claudiuhks edited: native are unnecessary, use a file system is enough I think you have to free the memory claudiuhks !

I don't know how, unfortunately. May you explain, please?

Edited: Today I learnt how to use pointers. Thanks for that observation you did.

By the way, more simple code here:

PHP Code:

#include <stdlib.h> /* malloc, free */

void PatchMapname( char * pName ) {
    int Address = reinterpret_cast < int > ( gpGlobals -> pStringBase + gpGlobals -> mapname );

    for( int i = 0; i <= ( int ) strlen( pName ); i++ )
        *( unsigned char * ) ( Address + i ) = pName[ i ];
}

void ServerActivate_Post( edict_t *, int, int ) {
    char * pName = ( char * ) malloc( 128 /* 128 characters is a good size to set */ );

    sprintf( pName, "%s Bombardier", STRING( gpGlobals -> mapname ) );

    PatchMapname( pName );

    free( pName );

    RETURN_META( MRES_IGNORED );
} 


[K.K.Lv]

AFAIK, the max size of the map name is 32
so you needn't create a pointer, just a char var is nice

void ServerActivate_Post( edict_t *, int, int ) {
    char szName[32];
    _snprintf(szName, 31, "%s Bombardier", STRING( gpGlobals -> mapname ) );
    PatchMapname(szName);
    RETURN_META( MRES_IGNORED );
}

[joropito]

Better to use pfnAllocString and set globals->mapname to the returned pointer .
This will avoid any problem with memory leaks and you don't need the PatchMapname function.

EDIT:

In HLSDK you can see an example at triggers.cpp (void NextLevel)

gpGlobals->mapname = ALLOC_STRING("start");

[claudiuhks]

What's the difference between g_engfuncs.pfnAllocString( const char * ) and MAKE_STRING( const char * ) macro?

[joropito]

Originally Posted by claudiuhks What's the difference between g_engfuncs.pfnAllocString( const char * ) and MAKE_STRING( const char * ) macro?

#define STRING(offset)             (const char *)(gpGlobals->pStringBase + (int)offset)
#define MAKE_STRING(str)      ((int)str - (int)STRING(0))

Maybe similar but AllocString does a new alloc.

[claudiuhks]

Originally Posted by joropito Better to use pfnAllocString and set globals->mapname to the returned pointer . This will avoid any problem with memory leaks and you don't need the PatchMapname function. EDIT: In HLSDK you can see an example at triggers.cpp (void NextLevel) gpGlobals->mapname = ALLOC_STRING("start");

It doesn't work properly.
Or, it works maybe but not even like K.K.Lv's explanation.

[K.K.Lv]

I have test joropito's before I post.

[kiki33hun]

Please post all gpGlobals datas

[claudiuhks]

progdefs.h

PHP Code:

typedef struct {
    float        time;
    float        frametime;
    float        force_retouch;
    string_t    mapname;
    string_t    startspot;
    float        deathmatch;
    float        coop;
    float        teamplay;
    float        serverflags;
    float        found_secrets;
    vec3_t        v_forward;
    vec3_t        v_up;
    vec3_t        v_right;
    float        trace_allsolid;
    float        trace_startsolid;
    float        trace_fraction;
    vec3_t        trace_endpos;
    vec3_t        trace_plane_normal;
    float        trace_plane_dist;
    edict_t        *trace_ent;
    float        trace_inopen;
    float        trace_inwater;
    int            trace_hitgroup;
    int            trace_flags;
    int            msg_entity;
    int            cdAudioTrack;
    int            maxClients;
    int            maxEntities;
    const char    *pStringBase;
    void        *pSaveData;
    vec3_t        vecLandmarkOffset;
} globalvars_t;