From cb9d0deb713f16956339f21af5e231dac63a9646 Mon Sep 17 00:00:00 2001 From: Vamsikrishna Bhagi Date: Wed, 3 Dec 2014 14:46:39 +0530 Subject: Bug #19688008 CREATE_ASYMMETRIC_PUB_KEY: CRASHES IN OPENSSL CODE Problem: UDF doesn't handle the arguments properly when they are of string type due to a misplaced break. The length of arguments is also not set properly when the argument is NULL. Solution: Fixed the code by putting the break at right place and setting the argument length to zero when the argument is NULL. --- sql/udf_example.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) (limited to 'sql/udf_example.c') diff --git a/sql/udf_example.c b/sql/udf_example.c index bfd153b7d28..af625b7ebf0 100644 --- a/sql/udf_example.c +++ b/sql/udf_example.c @@ -1,4 +1,4 @@ -/* Copyright (c) 2000, 2010, Oracle and/or its affiliates. All rights reserved. +/* Copyright (c) 2000, 2014, Oracle and/or its affiliates. All rights reserved. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -295,9 +295,20 @@ char *metaphon(UDF_INIT *initid __attribute__((unused)), if (!word) /* Null argument */ { + /* The length is expected to be zero when the argument is NULL. */ + assert(args->lengths[0] == 0); *is_null=1; return 0; } + else + { + /* + A check to confirm that argument length has the value it is + expected to have. + */ + assert(strlen(word) == args->lengths[0]); + } + w_end=word+args->lengths[0]; org_result=result; -- cgit v1.2.1