Libseccomp: बग: seccomp_arch_add () रिटर्न -EEXISTS एंडियन बेमेल पर

को निर्मित 20 जून 2017 · 18टिप्पणियाँ · स्रोत: seccomp/libseccomp

(ध्यान दें कि मेरे मुद्दे का विवरण गोलांग से शुरू होता है लेकिन यह वास्तव में एक सी मुद्दा है, नीचे देखें)

मैं आज कुछ यूनिट परीक्षण लिख रहा था जो मेरे amd64 सिस्टम पर ScmpFilter.AddArch(seccomp.ArchPPC) का प्रयोग करते हैं। इसने कोई त्रुटि नहीं लौटाई, हालांकि आर्किटेक्चर को फ़िल्टर में नहीं जोड़ा गया था (जैसा कि ExportPFC के माध्यम से दिखाई देता है)।

यहां एक छोटा पुनरुत्पादक है (amd64 या i386 पर चलाने की आवश्यकता है):

package main

import (
    "os"
    "github.com/seccomp/libseccomp-golang"
)

func main() {
    secFilter, err := seccomp.NewFilter(seccomp.ActKill)
    if err != nil {
        panic(err)
    }
    err = secFilter.AddArch(seccomp.ArchPPC)
    if err != nil {
        panic(err)
    }
    secFilter.ExportPFC(os.Stdout)
}

कुछ डिबगिंग के बाद यह पता चला कि seccomp_arch_add() एक एंडियन बेमेल होने पर EEXIST लौटाएगा। db.c:db_col_db_add() में है:

if (col->endian != 0 && col->endian != db->arch->endian)
        return -EFAULT;

गोलंग कोड (ठीक है) EEXIST को अनदेखा करता है जो मेरे द्वारा देखे गए व्यवहार की ओर जाता है।

मुझे आश्चर्य है कि क्या यह समझ में आता है कि seccomp_arch_add() एक अलग त्रुटि कोड लौटाएगा, शायद EINVAL या कुछ और? यदि यह बहुत खतरनाक है (क्योंकि यह मौजूदा अनुप्रयोगों को तोड़ सकता है), तो शायद इसे बेहतर तरीके से प्रलेखित किया जा सकता है। मुझे एक पीआर प्रदान करने में खुशी हो रही है।

bug prioritmedium

स्रोत

mvo5

सभी 18 टिप्पणियाँ

इसकी रिपोर्ट करने के लिए धन्यवाद, मुझे और करीब से देखना होगा।

pcmoore 21 जून 2017

मुझे बहुत खेद है कि मुझे इस @mvo5 पर वापस आने में इतना समय

चीजों को थोड़ा स्पष्ट करने के लिए, प्रासंगिक वर्तमान db.c:db_col_db_add() इस तरह दिखता है:

        if (col->endian != 0 && col->endian != db->arch->endian)
                return -EEXIST;

... मुझे लगता है कि मूल समस्या रिपोर्ट में उपरोक्त संस्करण समस्या को हल करने के लिए स्थानीय रूप से पैच की गई प्रतिलिपि थी। उस ने कहा, यहां दी गई त्रुटि को बदलना उचित लगता है; हम कम से कम कुछ अन्य आर्क/एंडियन कोड में EDOM का उपयोग कर रहे हैं, क्या यह आपको उचित लगता है?

आपको क्या लगता है @mheon?

ऐसा लगता है कि जब हम इस पर हैं तो हमें शायद db.c:db_col_merge() भी अपडेट करना चाहिए।

pcmoore 15 जन॰ 2019

👍1

एक एपीआई दृष्टिकोण से, मुझे निश्चित रूप से लगता है कि यह बदलाव करने के लिए अच्छी समझ में आता है - डिबगिंग के लिए त्रुटि कोड को ओवरलोड करना हमेशा समस्याग्रस्त होता है।

libseccomp-golang पक्ष पर, इसे किसी भी कोड परिवर्तन की आवश्यकता नहीं होनी चाहिए, जब तक कि नकारात्मक ERRNO सम्मेलन बनाए रखा जाता है; नई त्रुटि का अर्थ समझाने के लिए AddArch पर टिप्पणियों की कुछ पंक्तियाँ हो सकती हैं, इसलिए API दस्तावेज़ पूरा हो जाएगा।

mheon 15 जन॰ 2019

ट्रैविस अभी भी कर्नेल का बहुत पुराना चल रहा है, इस प्रकार परीक्षण # 47 फिर से विफल हो गया। जैसा कि @pcmoore ने कुछ समय पहले उल्लेख किया था, मैं इससे बचने के लिए परीक्षण में कुछ स्मार्ट जोड़ने का प्रयास करूंगा।

वरना इस बदलाव से बाकी सब कुछ मेरी किताब में अच्छा लगता है

drakenclimber 4 फ़र॰ 2019

@drakenclimber मुझे लगता है कि आपका मतलब टेस्ट 46 है, 47 नहीं, है ना?

कुछ महीने पहले मैंने लाइव परीक्षणों के लिए एपीआई स्तर की जांच जोड़ी, लेकिन मुझे नहीं लगता कि हमें बीपीएफ-सिम परीक्षणों के लिए इसकी आवश्यकता है, है ना? बीपीएफ-सिम टेस्ट साफ चल रहे थे ...?

pcmoore 4 फ़र॰ 2019

ट्रैविस निश्चित रूप से उपरोक्त रन के लिए परीक्षण #47 (KILL_PROCESS) पर टिका हुआ है। मैंने आज सुबह मास्टर के सिर पर भी यही विफलता देखी जब मैंने इसे एक अलग शाखा में एक विवेक जांच के रूप में धकेल दिया।

परीक्षण 47-लाइव-किल_प्रोसेस%%001-00001 परिणाम: विफलता 47-लाइव-किल_प्रोसेस 3 KILL_PROCESS rc=12

हमने इस बारे में कुछ समय पहले बात की थी, इसलिए मेरी याददाश्त धुंधली हो सकती है, लेकिन मुझे लगा कि ट्रैविस को KILL_PROCESS परीक्षण में समस्या है क्योंकि ट्रैविस का कर्नेल 4.14 से पुराना है जब उस सुविधा को पेश किया गया था।

या मैं पागल हूँ... और पूरी तरह से गलत याद कर रहा हूँ?

drakenclimber 5 फ़र॰ 2019

हम्म, क्या हम वही लॉग देख रहे हैं? मैं निर्माण को देख रहा हूं और नीचे लॉग इन कर रहा हूं:

... जो निम्नलिखित परिणाम दिखाते हैं (यहां केवल "सी" परीक्षणों की प्रतिलिपि बनाई गई है, "पायथन" परिणाम समान थे):

 batch name: 46-sim-kill_process
 test mode:  c
 test type:  bpf-sim
Test 46-sim-kill_process%%001-00001 result:   ERROR 46-sim-kill_process rc=12
Test 46-sim-kill_process%%002-00001 result:   ERROR 46-sim-kill_process rc=12
Test 46-sim-kill_process%%003-00001 result:   ERROR 46-sim-kill_process rc=12
Test 46-sim-kill_process%%004-00001 result:   ERROR 46-sim-kill_process rc=12
Test 46-sim-kill_process%%005-00001 result:   ERROR 46-sim-kill_process rc=12
Test 46-sim-kill_process%%006-00001 result:   ERROR 46-sim-kill_process rc=12
 batch name: 47-live-kill_process
 test mode:  c
 test type:  live
Test 47-live-kill_process%%001-00001 result:   SKIPPED (must specify live tests)

... और हां, पुराने कर्नेल में कुछ लाइव परीक्षणों में समस्या है, लेकिन इसे 9d4f7f69714d5af80309aa1b8a6d2c8300bb6730 में ठीक किया जाना चाहिए था।

FWIW, मास्टर शाखा पर अंतिम ट्रैविस बिल्ड साफ चला:

https://travis-ci.org/seccomp/libseccomp/builds/431949896

मैंने सिर्फ यह सत्यापित करने के लिए मास्टर शाखा का उपयोग करके एक नया निर्माण शुरू किया कि ट्रैविस के साथ सब कुछ "ठीक" है:

https://travis-ci.org/seccomp/libseccomp/jobs/488759868

pcmoore 5 फ़र॰ 2019

मैं स्वीकार करता हूं कि मैं अब _वास्तव में_ भ्रमित हूं। आपका लिंक निश्चित रूप से समस्या # 46 परीक्षण दिखाता है। लेकिन जब मैं ऊपरी दाएं कोने में "रॉ लॉग" लिंक पर क्लिक करता हूं, तो यह बताता है कि #47 विफल रहा। आपके द्वारा ऊपर सूचीबद्ध लिंक के लिए, यहां वह जगह है जहां कच्चे लॉग ने मुझे पुनर्निर्देशित किया है:

https://api.travis-ci.org/v3/job/488643923/log.txt

drakenclimber 5 फ़र॰ 2019

इसलिए करीब से देखने पर मुझे लगता है कि हम दोनों सही हैं।

46 `ERROR` लौटा रहा है जैसा आपने बताया। और #47 `FAILURE` लौटा रहा है (जो मैंने मूल रूप से खोजा था।)

और यह सारांश में भी दिखाई देता है:

Regression Test Summary
 tests run: 14090
 tests skipped: 114
 tests passed: 14090
 tests failed: 0
 tests errored: 12

Regression Test Summary
 tests run: 16
 tests skipped: 0
 tests passed: 14
 tests failed: 2
 tests errored: 0

drakenclimber 5 फ़र॰ 2019

यदि मैं 3.x कर्नेल में बूट करता हूं तो मैं अपने सिस्टम में ट्रैविससीआई की समस्याओं को पुन: उत्पन्न करने में सक्षम हूं। टेस्ट 46 में परेशानी हो रही है sys_chk_seccomp_action() । यह अंततः seccomp_init() को एक पूर्ण संदर्भ वापस परीक्षण में वापस करने का कारण बनता है।

मुझे लगता है कि टेस्ट 47 की समस्याएं समान हैं, लेकिन मैंने इसकी विफलता पथ की जांच नहीं की। (यद्यपि एपीआई को सत्यापित करने के लिए जोड़ा गया परिवर्तन @pcmoore इसे रोका जाना चाहिए था।

मुझे उत्सुकता होगी कि ट्रैविस पर इसका कारण क्या बदल गया है। क्या वे वास्तव में पुराने कर्नेल में वापस आ गए थे? हमारी तरफ से कुछ?

ऐसा लगता है कि इसे संभालने के लिए हमें बीपीएफ-सिम परीक्षणों में कुछ स्मार्ट जोड़ने की आवश्यकता होगी। क्या हम *.tests फ़ाइल में जोड़े गए API कॉलम की नकल करना चाहते हैं या पूरी तरह से कुछ और करना चाहते हैं?

drakenclimber 5 फ़र॰ 2019

हाँ, मैं वास्तव में उलझन में हूँ कि यह क्यों काम कर रहा था जैसा कि अभी नहीं है। उबंटू 14.xx इन दिनों वास्तव में पुराना है, मैं यह देखने जा रहा हूं कि ट्रैविस पर कोई और हालिया संस्करण उपलब्ध है या नहीं।

pcmoore 5 फ़र॰ 2019

ऐसा लगता है कि Ubuntu 16.04 (Xenial) उपलब्ध है, आइए कोशिश करते हैं कि ...

pcmoore 5 फ़र॰ 2019

कमिट 06f63ba691cb9df119c6759e8f0a150a2a9cbe69 हमें Ubuntu 16.04 तक टक्कर देता है। मैं इसे मास्टर शाखा में कर रहा हूं, पीआर के रूप में नहीं, क्योंकि मैं इस स्विच को मजबूर करना चाहता हूं; अगर बिल्ड टूट जाता है तो हम इसे ठीक कर देंगे।

https://travis-ci.org/seccomp/libseccomp/builds/489230929

pcmoore 5 फ़र॰ 2019

👍1

ऐसा लगता है कि निर्माण ने परीक्षणों के साथ समस्या को ठीक कर दिया था, लेकिन क्लैंग को त्रुटि प्रबंधन कोड पथ में स्मृति रिसाव मिला। मैं इसे एक मिनट में ठीक कर दूंगा।

pcmoore 5 फ़र॰ 2019

बहुत बढ़िया! मदद के लिए धन्यवाद, @pcmoore

drakenclimber 5 फ़र॰ 2019

ठीक है, प्रतिबद्ध f8854f990004e71ccb9955c33d88d82cdb97ea42 के साथ हमारे पास एक स्वच्छ भवन मास्टर शाखा होनी चाहिए। इसने मेरी निजी शाखा पर ठीक काम किया, अब मुख्य निर्माण की प्रतीक्षा कर रहा है।

pcmoore 5 फ़र॰ 2019

@drakenclimber मैं देख रहा हूं कि आपने ऊपर दिए गए लॉग में इसके लिए एक पैच तैयार कर लिया है, लेकिन मुझे अभी तक कोई पीआर नहीं दिख रहा है - क्या आप अभी भी पैच के साथ कुछ मुद्दों का पीछा कर रहे हैं या यह पीआर के लिए तैयार है?

pcmoore 18 फ़र॰ 2019

इसे कमिट 4a35b6ea6f7c836734536420c50a2745a9e24c69 के साथ हल किया जाना चाहिए, इसे अभी बंद करें। अगर किसी को इसमें कोई समस्या मिलती है तो कृपया इस मुद्दे को फिर से खोलें या एक नया बनाएं।

pcmoore 18 फ़र॰ 2019

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Libseccomp: बग: seccomp_arch_add () रिटर्न -EEXISTS एंडियन बेमेल पर

सभी 18 टिप्पणियाँ

46 ERROR लौटा रहा है जैसा आपने बताया। और #47 FAILURE लौटा रहा है (जो मैंने मूल रूप से खोजा था।)

संबंधित मुद्दों

46 `ERROR` लौटा रहा है जैसा आपने बताया। और #47 `FAILURE` लौटा रहा है (जो मैंने मूल रूप से खोजा था।)