otland/forgottenserver

Missing isCorpse check at ProtocolGame::sendContainer

Open

#3.855 aberto em 16 de dez. de 2021

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C++ (1.145 forks)auto 404
featuregood first issuepriority: low

Métricas do repositório

Stars
 (1.813 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Before creating an issue, please ensure:

  • This is a bug in the software that resides in this repository, and not a support matter (use https://otland.net/forums/support.16/ for support)
  • This issue is reproducible without changes to the C++ code in this repository

Steps to reproduce (include any configuration/script required to reproduce)

  1. Kill a monster
  2. Open its corpse
  3. Move any item to inside the corpse

Expected behaviour

Should not be possible, a red square will appear in the slot you are trying to move it to

Actual behaviour

You can add any item to monster corpses

Explanation

I don't know since when it was added, but its there on official, and I think it was done to prevent some analyzers to not contain misinformation

Related code

here is the part that need a isCorpse() check

PS

Make it configurable, but enabled by default

Guia do colaborador

Direção de pesquisa
Pilha de tecnologia
Domain
Tipo Issue
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Prerequisites
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.