Jump to content
hemagx

Odin Server side manner

Recommended Posts

errr ... there are so many error during compile, because I didn't off that stupid microsoft warning ...

 

try change those

sprintf -> safesnprintf

strncpy -> safestrncpy

 

the only thing that I don't think its possible is sscanf and fopen

strcmpi ... no idea

.

if(retVal == 0 || !sd) // sd missing case is impossible, but who knows... it's RAGNAROK!
use nullpo in this case

nullpo_retr(0, sd);

 

EDIT: is there a need to call a new struct for pcre and pcre_extra ?

when I try mine, I don't need to use them

http://upaste.me/67672221608816857

Edited by AnnieRuru

Share this post


Link to post
Share on other sites

1- Change sprintf/strncpy

possible next version or if they got not working before make new version, otherwise i see no reason to do special version for it.

 

2- it's same, i prefer it this way for my own work.

 

3- yes, i have to save them compiled/optimized  or i will have to do it each time the player send message.

Share this post


Link to post
Share on other sites

1 & 2

ok I add disable 4996 warning, instantly no more error

http://herc.ws/wiki/Building_HPM_Plugin_for_MSVC#Step_18

I just not sure to add this as Step-18 ? or an additional step ...

because I found this in our sample and plugin property page, and after google around many people also add them

http://stackoverflow.com/questions/3317536/visual-studio-warning-c4996?lq=1

 

3,

oh I always call new struct ... no wonder, I guess your method does optimized a bit

Share this post


Link to post
Share on other sites

many many members keep reporting this error, ever since I started write plugin

http://herc.ws/board/topic/7242-market-clone/?p=44110

http://herc.ws/board/topic/5057-auraset/?p=38152

it wasn't just me

 

that 4996 error has been ignore by sample/hook ... but if you add your own plugin, of course it spit errors

Share this post


Link to post
Share on other sites

many many members keep reporting this error, ever since I started write pluginhttp://herc.ws/board/topic/7242-market-clone/?p=44110http://herc.ws/board/topic/5057-auraset/?p=38152it wasn't just methat 4996 error has been ignore by sample/hook ... but if you add your own plugin, of course it spit errors

Because its just stupid msvc, trying to compile as c++

(For me, I just lower down the warning level)

Share this post


Link to post
Share on other sites

eh, make sure you copy sample settings i guess ?

i'm using MSVC for testing :) and i remember very well Hercules disabled those.

Share this post


Link to post
Share on other sites

eh, make sure you copy sample settings i guess ?

i'm using MSVC for testing :) and i remember very well Hercules disabled those.

 

great plugin dude! by the way it seems like those options are not working (I'm using the full version by the way)

 

 

// Do you want GM above this level bypass the filter ?
// set to any level you want to disable set it to 100
gm_min_level: 99


// After how much times the player say bad words you want to mute it ?
// Set it to 0 to disable
mute_after: 3
 

even with my gm character (group id/gm lvl 99) it still can't bypass anything, just ******

and I've tried with a normal account (lvl 0) and tried spamming the word for like 10 times, it just ****** but doesn't mutes me :(

Edited by evilpuncker

Share this post


Link to post
Share on other sites

If its shown as *****, probably your client side manner is acting, and *** shown means client censored the words and ACTUALLY Sended those asterisks to the server, so this plugin won't be able to do anything when the server haven't received the correct word

Share this post


Link to post
Share on other sites

If its shown as *****, probably your client side manner is acting, and *** shown means client censored the words and ACTUALLY Sended those asterisks to the server, so this plugin won't be able to do anything when the server haven't received the correct word

 

 

indeed, my bad then  :P  :P it is working, the only thing that I noticed is that there is no red bubble being show, maybe hercules bug?

Share this post


Link to post
Share on other sites

I tried to fix it by myself but no success :( this plugin is so awesome

 

You're lucky that i'm facing serious issue right now and i can't continue what i'm working on.

Gonna work on this plugin now :)

Share this post


Link to post
Share on other sites

 

I tried to fix it by myself but no success :( this plugin is so awesome

 

You're lucky that i'm facing serious issue right now and i can't continue what i'm working on.

Gonna work on this plugin now :)

 

glad to hear that <3

Share this post


Link to post
Share on other sites

 

 

I tried to fix it by myself but no success :( this plugin is so awesome

 

You're lucky that i'm facing serious issue right now and i can't continue what i'm working on.

Gonna work on this plugin now :)

 

glad to hear that <3

 

got few issue, gonna delay it...(hears someone crying in the corner)..

Share this post


Link to post
Share on other sites

I was able to make the lite version work, except that I don't know what to return in place of return retVal;... I just changed it to return true but I don't know if its the correct way :D

 

here is the "working" lite version in case anyone wants! unfortunately, the full version was too complicated for me to try fixing it

 

 

WARNING: players can bypass it by simply using an uppercase letter, since the check is not case sensitive and the filter doesn't seem to support "/i" to ignore case sensitiveness

Edited by evilpuncker

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.