mirror of
https://github.com/samba-team/samba.git
synced 2025-01-11 05:18:09 +03:00
r24753: Allow host name in binding string without transport.
This commit is contained in:
parent
2a5a0819ee
commit
f7051d3a84
@ -28,9 +28,9 @@
|
||||
#include "librpc/ndr/libndr.h"
|
||||
|
||||
enum dcerpc_transport_t {
|
||||
NCACN_NP, NCACN_IP_TCP, NCACN_IP_UDP, NCACN_VNS_IPC, NCACN_VNS_SPP,
|
||||
NCACN_AT_DSP, NCADG_AT_DDP, NCALRPC, NCACN_UNIX_STREAM, NCADG_UNIX_DGRAM,
|
||||
NCACN_HTTP, NCADG_IPX, NCACN_SPX };
|
||||
NCA_UNKNOWN, NCACN_NP, NCACN_IP_TCP, NCACN_IP_UDP, NCACN_VNS_IPC,
|
||||
NCACN_VNS_SPP, NCACN_AT_DSP, NCADG_AT_DDP, NCALRPC, NCACN_UNIX_STREAM,
|
||||
NCADG_UNIX_DGRAM, NCACN_HTTP, NCADG_IPX, NCACN_SPX };
|
||||
|
||||
/*
|
||||
this defines a generic security context for signed/sealed dcerpc pipes.
|
||||
|
@ -235,15 +235,17 @@ char *dcerpc_binding_string(TALLOC_CTX *mem_ctx, const struct dcerpc_binding *b)
|
||||
{
|
||||
char *s = talloc_strdup(mem_ctx, "");
|
||||
int i;
|
||||
const char *t_name=NULL;
|
||||
const char *t_name = NULL;
|
||||
|
||||
for (i=0;i<ARRAY_SIZE(transports);i++) {
|
||||
if (transports[i].transport == b->transport) {
|
||||
t_name = transports[i].name;
|
||||
if (b->transport != NCA_UNKNOWN) {
|
||||
for (i=0;i<ARRAY_SIZE(transports);i++) {
|
||||
if (transports[i].transport == b->transport) {
|
||||
t_name = transports[i].name;
|
||||
}
|
||||
}
|
||||
if (!t_name) {
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
if (!t_name) {
|
||||
return NULL;
|
||||
}
|
||||
|
||||
if (!GUID_all_zero(&b->object.uuid)) {
|
||||
@ -251,8 +253,13 @@ char *dcerpc_binding_string(TALLOC_CTX *mem_ctx, const struct dcerpc_binding *b)
|
||||
GUID_string(mem_ctx, &b->object.uuid));
|
||||
}
|
||||
|
||||
s = talloc_asprintf_append(s, "%s:", t_name);
|
||||
if (!s) return NULL;
|
||||
if (t_name != NULL) {
|
||||
s = talloc_asprintf_append(s, "%s:", t_name);
|
||||
if (s == NULL)
|
||||
return NULL;
|
||||
} else {
|
||||
s = NULL;
|
||||
}
|
||||
|
||||
if (b->host) {
|
||||
s = talloc_asprintf_append(s, "%s", b->host);
|
||||
@ -323,27 +330,29 @@ NTSTATUS dcerpc_parse_binding(TALLOC_CTX *mem_ctx, const char *s, struct dcerpc_
|
||||
b->object.if_version = 0;
|
||||
|
||||
p = strchr(s, ':');
|
||||
if (!p) {
|
||||
return NT_STATUS_INVALID_PARAMETER;
|
||||
}
|
||||
|
||||
type = talloc_strndup(mem_ctx, s, PTR_DIFF(p, s));
|
||||
if (!type) {
|
||||
return NT_STATUS_NO_MEMORY;
|
||||
}
|
||||
|
||||
for (i=0;i<ARRAY_SIZE(transports);i++) {
|
||||
if (strcasecmp(type, transports[i].name) == 0) {
|
||||
b->transport = transports[i].transport;
|
||||
break;
|
||||
if (p == NULL) {
|
||||
b->transport = NCA_UNKNOWN;
|
||||
} else {
|
||||
type = talloc_strndup(mem_ctx, s, PTR_DIFF(p, s));
|
||||
if (!type) {
|
||||
return NT_STATUS_NO_MEMORY;
|
||||
}
|
||||
|
||||
for (i=0;i<ARRAY_SIZE(transports);i++) {
|
||||
if (strcasecmp(type, transports[i].name) == 0) {
|
||||
b->transport = transports[i].transport;
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
if (i==ARRAY_SIZE(transports)) {
|
||||
DEBUG(0,("Unknown dcerpc transport '%s'\n", type));
|
||||
return NT_STATUS_INVALID_PARAMETER;
|
||||
}
|
||||
}
|
||||
if (i==ARRAY_SIZE(transports)) {
|
||||
DEBUG(0,("Unknown dcerpc transport '%s'\n", type));
|
||||
return NT_STATUS_INVALID_PARAMETER;
|
||||
}
|
||||
|
||||
s = p+1;
|
||||
s = p+1;
|
||||
}
|
||||
|
||||
p = strchr(s, '[');
|
||||
if (p) {
|
||||
|
@ -95,6 +95,27 @@ static const char *test_strings[] = {
|
||||
"ncacn_unix_stream:[/tmp/epmapper,sign]",
|
||||
};
|
||||
|
||||
static bool test_no_transport(struct torture_context *tctx)
|
||||
{
|
||||
const char *binding = "somehost";
|
||||
struct dcerpc_binding *b;
|
||||
const char *s;
|
||||
|
||||
/* Parse */
|
||||
torture_assert_ntstatus_ok(tctx, dcerpc_parse_binding(tctx, binding, &b),
|
||||
"Error parsing binding string");
|
||||
|
||||
torture_assert(tctx, b->transport == NCA_UNKNOWN, "invalid transport");
|
||||
|
||||
s = dcerpc_binding_string(tctx, b);
|
||||
torture_assert(tctx, s != NULL, "Error converting binding back to string");
|
||||
|
||||
torture_assert_casestr_equal(tctx, binding, s,
|
||||
"Mismatch while comparing original and regenerated binding strings");
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
struct torture_suite *torture_local_binding_string(TALLOC_CTX *mem_ctx)
|
||||
{
|
||||
int i;
|
||||
@ -106,5 +127,7 @@ struct torture_suite *torture_local_binding_string(TALLOC_CTX *mem_ctx)
|
||||
test_BindingString, test_strings[i]);
|
||||
}
|
||||
|
||||
torture_suite_add_simple_test(suite, "no transport", test_no_transport);
|
||||
|
||||
return suite;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user