fortra/impacket

Unsupported data type on TDS version >=7.2

Open

#877 ouverte le 9 juin 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Python (3 421 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (12 244 stars)
Métriques de merge PR
 (Merge moyen 30j 8h) (10 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Inspectez la gestion de la version TDS dans tds.py, en particulier la methode parseColMetaData. Ajustez la taille du champ userType en fonction de la version TDS : 2 octets pour la version <7.2, 4 octets pour la version >=7.2. Tracez la logique de detection de version pour garantir un comportement correct.
Stack technique
python
Domaine
backendapinetworking
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70