weird value printed when using function returning

[rajatjindal]

Hello again,

I am trying to use wasm-tools-go to generate bindings for the following wit:

interface variables {
    /// Get an application variable value for the current component.
    ///
    /// The name must match one defined in in the component manifest.
    get: func(name: string) -> result<string, error>;

    /// The set of errors which may be raised by functions in this interface.
    variant error {
        /// The provided variable name is invalid.
        invalid-name(string),
        /// The provided variable is undefined.
        undefined(string),
        /// A variables provider specific error has occurred.
        provider(string),
        /// Some implementation-specific error has occurred.
        other(string),
    }
}

when using the bindings generated, I see weird value printed:

key: password, val is "\x00\x00\x00\x00\x00\x00"

pseudo code:

func Get(key string) (string, error) {
	result := variables.Get(key)
	if result.IsErr() {
		return "", errorVariantToError(*result.Err())
	}

	stdout.GetStdout().Write(cm.ToList([]byte(fmt.Sprintf("key: %s, val is %q \n", key, *result.OK()))))

	return *result.OK(), nil
}

I printed the value on host component implementation side, and it seems to be printing and returning the correct value.

any pointers what could be wrong here? thank you so much for all the help so far.

[ydnar]

Hi there! Can you try with the latest version of wit-bindgen-go?

[rajatjindal]

Hi @ydnar, I tried this again today, and still running into same prob. I am adding some more details incase if that helps narrow down the problem:

wit file:

interface variables {
    /// Get an application variable value for the current component.
    ///
    /// The name must match one defined in in the component manifest.
    get: func(name: string) -> result<string, error>;

    /// The set of errors which may be raised by functions in this interface.
    variant error {
        /// The provided variable name is invalid.
        invalid-name(string),
        /// The provided variable is undefined.
        undefined(string),
        /// A variables provider specific error has occurred.
        provider(string),
        /// Some implementation-specific error has occurred.
        other(string),
    }
}

generated code (ommitting error functions):

// Get represents the imported function "get".
//
// Get an application variable value for the current component.
//
// The name must match one defined in in the component manifest.
//
//	get: func(name: string) -> result<string, error>
//
//go:nosplit
func Get(name string) (result cm.ErrResult[string, Error]) {
	name0, name1 := cm.LowerString(name)
	wasmimport_Get((*uint8)(name0), (uint32)(name1), &result)
	return
}

//go:wasmimport fermyon:spin/variables@2.0.0 get
//go:noescape
func wasmimport_Get(name0 *uint8, name1 uint32, result *cm.ErrResult[string, Error])

When I print this inside host component:

tracing::trace!("variables get is {:?}", val);

## above prints:
# variables get is Ok("works2")

When I print this inside my guest code function:

func Get(key string) (string, error) {
	result := variables.Get(key)
	if result.IsErr() {
		return "", errorVariantToError(*result.Err())
	}

	val := result.OK()
	println(fmt.Sprintf("variables get inside sdk is %#v", *val))
	return *result.OK(), nil
}


func println(msg string) {
	stdout.GetStdout().Write(cm.ToList([]byte(msg)))
	stdout.GetStdout().Write(cm.ToList([]byte("\n")))
}

## above prints
# variables get inside sdk is \"\\x00\\x00\\x00\\x00\\x00\\x00\"

Thank you

[rajatjindal]

I realize my last comment is almost same as the original description. I will try to narrow down this to a small example, but if in the meantime you have any ideas that I could try, kindly let me know. thank you again.

[rajatjindal]

(kindly ignore me if this is incorrect)

do we need to lower/lift the string in the result here?

[ydnar]

Call OK() on the result to extract the string.

[rajatjindal]

I did call .OK() on the result (kindly refer to the partial code below), but still get same value

val := result.OK()
println(fmt.Sprintf("variables get inside sdk is %#v", *val))

[ydnar]

Can you use %s? Val should be *string.

[rajatjindal]

Hi Randy, thank you for your continued feedback on this. I tried this as well, and see following:

variables get inside sdk is \0\0\0\0\0\0\0

I should mention that the number of \0 here (or \x00 that I mentioned earlier) is exactly the same as number of characters that I am expecting as output. It seems like its just the content which is not correct.

[ydnar]

1. Have you successfully been able to have an imported function return a string?
2. Is the length of the returned string correct (7 bytes)? It’s just returning blank memory?
3. Can you check the value of the unsafe.StringData pointer of the returned string?
4. What implements the get() function?

[rajatjindal]

Hi Randy,

I just tried this with a host-component that implements following wit:

    /// Return a list of all the keys in kv store
    get-keys: func() -> result<list<string>, error>;

and the result I receive back is (the number of items are same as the unique keys available in the kv store):

[]string{"", ""}

These host components that I am trying out are implemented as part of fermyon/spin

e.g. variables

and

kv store

[rajatjindal]

I will be trying out the other suggestions and will report back the result 👍

[rajatjindal]

on trying to print stringdata, I get following:

stringdata is (*uint8)(0x20)

^^ is the output of following code:

// Get an application variable value for the current component.
//
// The name must match one defined in in the component manifest.
func Get(key string) (string, error) {
	result := variables.Get(key)
	if result.IsErr() {
		return "", errorVariantToError(*result.Err())
	}

	val := result.OK()

	//check the value of the unsafe.StringData pointer of the returned string
	stringdata := unsafe.StringData(*val)

	stdout.GetStdout().Write(cm.ToList([]byte(fmt.Sprintf("stringdata is %#v \n", stringdata))))
	return *result.OK(), nil
}

variables.Get is the generated code:

//go:nosplit
func Get(name string) (result cm.ErrResult[string, Error]) {
	name0, name1 := cm.LowerString(name)
	wasmimport_Get((*uint8)(name0), (uint32)(name1), &result)
	return
}

//go:wasmimport fermyon:spin/variables@2.0.0 get
//go:noescape
func wasmimport_Get(name0 *uint8, name1 uint32, result *cm.ErrResult[string, Error])

[rajatjindal]

I have also created a small repro repo for this: https://github.com/rajatjindal/wasip2-string

it needs to run using spin (I can try to get a reproducible scenario with wasmtime if you think that would be better).

[ydnar]

Thanks. I assume this is run with the wasip2 fork of TinyGo?

[rajatjindal]

yes, that is right. I updated the repo in last day or so.