Flummi/amxmodx
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #277 from Arkshine/fix/potential-issues

Browse Source 
Fix some potential issues
This commit is contained in:
Vincent Herbet 2015-08-10 15:28:32 +02:00
commit 4563f56e77
4 changed files with 16 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
amxmodx/amxmodx.cpp
View File

@ -1423,19 +1423,19 @@ static cell AMX_NATIVE_CALL get_plugin(AMX *amx, cell *params) /* 11 param */
set_amxstring(amx, params[8], a->getAuthor(), params[9]); set_amxstring(amx, params[8], a->getAuthor(), params[9]);
set_amxstring(amx, params[10], a->getStatus(), params[11]); set_amxstring(amx, params[10], a->getStatus(), params[11]);
if (params[0] / sizeof(cell) >= 12)
{
cell *jit_info = get_amxaddr(amx, params[12]);
#if defined AMD64 || !defined JIT
*jit_info = 0;
#else
*jit_info = a->isDebug() ? 0 : 1;
#endif
}
return a->getId(); return a->getId();
} }
if (params[0] / sizeof(cell) >= 12)
{
cell *jit_info = get_amxaddr(amx, params[12]);
#if defined AMD64 || !defined JIT
*jit_info = 0;
#else
*jit_info = a->isDebug() ? 0 : 1;
#endif
}
return -1; return -1;
} }

2
modules/dod/dodfun/CMisc.h
View File

@ -280,7 +280,7 @@ class CObjective {
public: public:
int count; int count;
objinfo_t obj[12]; objinfo_t obj[12];
inline void Clear() { count = 0; memset(obj,0,sizeof(objinfo_s)); } inline void Clear() { count = 0; memset(obj,0,sizeof(obj)); }
void SetKeyValue( int index, char *keyname, char *value ); void SetKeyValue( int index, char *keyname, char *value );
void InitObj(int dest = MSG_ALL , edict_t* ed = NULL); void InitObj(int dest = MSG_ALL , edict_t* ed = NULL);

5
modules/dod/dodx/usermsg.cpp
View File

@ -152,10 +152,13 @@ void Client_Health_End(void* mValue)
if ( !isModuleActive() ) if ( !isModuleActive() )
return; return;
if (!mPlayer)
return;
edict_t *enemy = mPlayer->pEdict->v.dmg_inflictor; edict_t *enemy = mPlayer->pEdict->v.dmg_inflictor;
int damage = (int)mPlayer->pEdict->v.dmg_take; int damage = (int)mPlayer->pEdict->v.dmg_take;
if ( !mPlayer || !damage || !enemy ) if (!damage || !enemy)
return; return;
int weapon = 0; int weapon = 0;

2
modules/regex/CRegEx.cpp
View File

@ -583,7 +583,7 @@ int RegEx::Replace(char *text, size_t textMaxLen, const char *replace, size_t re
*/ */
if (flags & PCRE_DUPNAMES) if (flags & PCRE_DUPNAMES)
{ {
memset(ovector, 0, REGEX_MAX_SUBPATTERNS); memset(ovector, 0, REGEX_MAX_SUBPATTERNS * sizeof(int));
/** /**
* pcre_copy_named_substring needs a vector containing sub-patterns ranges * pcre_copy_named_substring needs a vector containing sub-patterns ranges