[DiscordArchive] ```cpp
[DiscordArchive] ```cpp
Original source
```cpp
// caster is Player*
// target is Item*
caster->DestroyItem(target->GetBagSlot(), target->GetSlot(), true);
```
Can someone tell me why the above code crashes my worldserver on the next `SaveInventoryToDB()` call?
I have tried to look around on how the core uses `DestroyItem` in other places, and they all seem to do the same as I?
rektbyfaith
08-01-2020, 03:41 PM #1Archived author: caweren • Posted: 2025-02-10T16:30:05.660000+00:00
Original source
```cpp
// caster is Player*
// target is Item*
caster->DestroyItem(target->GetBagSlot(), target->GetSlot(), true);
```
Can someone tell me why the above code crashes my worldserver on the next `SaveInventoryToDB()` call?
I have tried to look around on how the core uses `DestroyItem` in other places, and they all seem to do the same as I?
Original source
<:wHaT:585568060992126976>
rektbyfaith
08-01-2020, 03:42 PM #2Archived author: MaxtorCoder • Posted: 2020-08-01T15:42:07.783000+00:00
Original source
<:wHaT:585568060992126976>
Original source
I must be tired <:KEKW:715590037235957890>
rektbyfaith
08-01-2020, 03:42 PM #3Archived author: MaxtorCoder • Posted: 2020-08-01T15:42:21.168000+00:00
Original source
I must be tired <:KEKW:715590037235957890>
Original source
If it's just snprintf under the hood there's a way to break it and have it incorrectly parse a string as an integer and lead to SQL injection
rektbyfaith
08-01-2020, 03:42 PM #4Archived author: Harbynger • Posted: 2020-08-01T15:42:42.559000+00:00
Original source
If it's just snprintf under the hood there's a way to break it and have it incorrectly parse a string as an integer and lead to SQL injection
Original source
Look at how `PQuery` works <:pepocozy:638459688807301141>
rektbyfaith
08-01-2020, 03:42 PM #5Archived author: MaxtorCoder • Posted: 2020-08-01T15:42:57.929000+00:00
Original source
Look at how `PQuery` works <:pepocozy:638459688807301141>
Original source
And yes, honestly that is a bad way,.
rektbyfaith
08-01-2020, 03:43 PM #6Archived author: MaxtorCoder • Posted: 2020-08-01T15:43:02.184000+00:00
Original source
And yes, honestly that is a bad way,.
Original source
yeah that's what im saying, i dont know what it does, but that'd be what i'd personally worry about
rektbyfaith
08-01-2020, 03:43 PM #7Archived author: Harbynger • Posted: 2020-08-01T15:43:23.733000+00:00
Original source
yeah that's what im saying, i dont know what it does, but that'd be what i'd personally worry about
Original source
Maybe its fine
rektbyfaith
08-01-2020, 03:43 PM #8Archived author: Harbynger • Posted: 2020-08-01T15:43:29.755000+00:00
Original source
Maybe its fine
Original source
but the whole ahead-of-time variable declaration spam seems like someone trying really hard to write K&R but it's C++
rektbyfaith
08-01-2020, 03:44 PM #9Archived author: Harbynger • Posted: 2020-08-01T15:44:17.743000+00:00
Original source
but the whole ahead-of-time variable declaration spam seems like someone trying really hard to write K&R but it's C++
Original source
we have more than 4kb memory now, no need for that stuff anymore
rektbyfaith
08-01-2020, 03:44 PM #10Archived author: Harbynger • Posted: 2020-08-01T15:44:39.670000+00:00
Original source
we have more than 4kb memory now, no need for that stuff anymore
1 Guest(s)
1 Guest(s)