fortra/impacket

Unsupported data type on TDS version >=7.2

Open

#877 aberto em 9 de jun. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Python (3.421 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (12.244 stars)
Métricas de merge de PR
 (Mesclagem média 30d 8h) (10 fundiu PRs em 30d)

Description

Configuration

impacket version: 0.9.21 Python version: 3.7.7 Target OS: Ubuntu 18.04

Debug Output With Command String

i.e.
smbexec -debug domain/user:password@127.0.0.1

>>> d = tds.TDS_COLMETADATA(b"\x81\x01\x00\x00\x00\x00\x00\x08\x00\x38\x09\x41\x00\x70\x00\x70\x00\x74\x00\x43\x00\x6f\x00\x6c\x00\x6f\x00\x72\x00")
>>> tds.MSSQL('bla').parseColMetaData(d)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/user/.local/lib/python3.7/site-packages/impacket/tds.py", line 1439, in parseColMetaData
    raise Exception("Unsupported data type: 0x%x" % colType)
Exception: Unsupported data type: 0x8

Additional context

This Column Metadata bytes was taken from pcap of TDS version 7.2.

As far as I understand, the problem is size of field userType:

  • until TDS version 7.1, userType size is 2 bytes.
  • since TDS version 7.2, userType size is 4 bytes.

The code of the library assume userType size is 2 bytes.

Guia do colaborador

Direção de pesquisa
Inspecione o tratamento da versão TDS no tds.py, especialmente o método parseColMetaData. Ajuste o tamanho do campo userType com base na versão TDS: 2 bytes para versão <7.2, 4 bytes para versão >=7.2. Rastreie a lógica de detecção de versão para garantir o comportamento correto.
Pilha de tecnologia
python
Domain
backendapinetworking
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70